Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for sponsorship packages #26926

Merged
merged 5 commits into from
Feb 29, 2024
Merged

Add support for sponsorship packages #26926

merged 5 commits into from
Feb 29, 2024

Conversation

davidfurey
Copy link
Member

@davidfurey davidfurey commented Feb 23, 2024

What is the value of this and can you measure success?

Allows US advertising team to sell sponsorships with "Advertising partner" and "Exclusive advertising partner" wording. See guardian/tagmanager#508 for more information.

This mainly involves library bumps, but since the CAPI library hasn't been bumped since the recipe atom was deprecated and the recipe element created, there are unfortunately a couple of commits to tidy that up that are unrelated to my main goal. See guardian/content-atom#161

Copy link
Contributor

@fredex42 fredex42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 This does not impact any work that we are currently doing as far as I can see

@cemms1
Copy link
Contributor

cemms1 commented Feb 28, 2024

Thank you for this @davidfurey! This really helped out me and @frederickobrien as we needed to test a new version of the CAPI client on #26931, which due to the breaking changes of v20 and v21 would have been much more work if we hadn't been able to rebase from this branch 🎉

Copy link
Member

@arelra arelra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@davidfurey davidfurey merged commit 01c800b into main Feb 29, 2024
3 checks passed
@davidfurey davidfurey deleted the sponsorship-package branch February 29, 2024 13:15
@prout-bot
Copy link
Collaborator

Seen on ADMIN-PROD (merged by @davidfurey 12 minutes and 14 seconds ago)

@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD (merged by @davidfurey 13 minutes and 18 seconds ago)

@deedeeh
Copy link
Contributor

deedeeh commented May 13, 2024

Just for information here is the upstream commercial-shared PR guardian/commercial-shared#59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants