-
Notifications
You must be signed in to change notification settings - Fork 31
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
Remove network fronts banner ads ab test #9047
Conversation
Size Change: +9 B (0%) Total Size: 1.44 MB ℹ️ View Unchanged
|
22433b8
to
604c660
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there are more occurrences of isInNetworkFrontsBannerTest
that should be removed or renamed
Thanks Daniel, I'll make sure to remove the other references. This PR is on hold for now as some folks are using the 0% test to do some of their own testing, so I'll add the "do not merge" label for now, then pick this up again later. |
604c660
to
810ae84
Compare
810ae84
to
23b84bf
Compare
What does this change?
fronts-banner
ad slot.Why?
fronts-banner
ad slot needs to be collapsed when collapsing the ad, therefore it needs a classname so that it can be found easily in the DOM.