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

Show ads on mobile discussion #26899

Merged
merged 10 commits into from
Feb 22, 2024
Merged

Show ads on mobile discussion #26899

merged 10 commits into from
Feb 22, 2024

Conversation

deedeeh
Copy link
Contributor

@deedeeh deedeeh commented Feb 12, 2024

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

Ads will be added to the comments section in DCR articles on mobile devices. This should provide an increase in revenue for mobile pageviews. This feature has been added behind a switch, so that we can turn it off if needed.

What does this change?

This PR is part of Commercial OKR work. Check these PRs for more information guardian/commercial#1239 and guardian/dotcom-rendering#10448

@emma-imber emma-imber force-pushed the dina/show-ads-on-mobile-discussion branch from fa32e3d to 7b37c10 Compare February 19, 2024 16:11
@emma-imber emma-imber changed the title Show ads on mobile discussion AB tests Show ads on mobile discussion Feb 19, 2024
@emma-imber emma-imber force-pushed the dina/show-ads-on-mobile-discussion branch from 9e56be0 to a6d4de5 Compare February 21, 2024 10:56
@deedeeh deedeeh marked this pull request as ready for review February 21, 2024 12:08
@deedeeh deedeeh requested a review from a team as a code owner February 21, 2024 12:08
@deedeeh deedeeh merged commit 476e04e into main Feb 22, 2024
3 checks passed
@deedeeh deedeeh deleted the dina/show-ads-on-mobile-discussion branch February 22, 2024 11:01
@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD, ADMIN-PROD (merged by @deedeeh 13 minutes and 4 seconds ago)

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.

None yet

5 participants