Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Revert "Closes #28417: Add nimbus flags for pre permission prompt" #28484

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

rocketsroger
Copy link
Contributor

Confirmed with Product that notification prompt doesn't not need a Nimbus experiment.
Reverts #28418

@rocketsroger rocketsroger requested review from a team as code owners January 10, 2023 17:24
Copy link
Contributor

@rahulsainani rahulsainani left a comment

Choose a reason for hiding this comment

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

@rahulsainani rahulsainani added the pr:approved PR that has been approved label Jan 10, 2023
@rocketsroger rocketsroger added the pr:needs-landing PRs that are ready to land [Will be merged by Mergify] label Jan 10, 2023
@mergify mergify bot merged commit 8cad8cb into main Jan 10, 2023
@bors bors bot deleted the revert-28418-28417 branch January 10, 2023 19:20
rocketsroger added a commit to rocketsroger/fenix that referenced this pull request Jan 12, 2023
rocketsroger added a commit to rocketsroger/fenix that referenced this pull request Jan 13, 2023
JohanLorenzo pushed a commit to mozilla-releng/staging-firefox-android that referenced this pull request Jan 25, 2023
@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Feb 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged pr:approved PR that has been approved pr:needs-landing PRs that are ready to land [Will be merged by Mergify]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants