-
Notifications
You must be signed in to change notification settings - Fork 925
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
Brave Together => Brave Talk changes - (uplift to 1.28.x) #9608
Conversation
Update Brave Talk Links
Rename Brave Together as Brave Talk on NTP (widget)
Brave Talk code cleanup (renaming) and putting behind flag for Griffin
With Once the Griffin change is live, QA will be able to run through brave/brave-browser#17210. CCing @stephendonner |
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.
Uplift into 1.28.x
approved after deliberating with @brave/uplift-approvers. QA has verified the PR on Nightly as per brave/brave-browser#16799 (comment), #9581 (comment), brave/brave-browser#16798 (comment) & #9605 (comment). @stephendonner also checked both main
(staging) & production
(release).
Uplift of #9468
Fixes brave/brave-browser#16799
Uplift of #9581
Fixes brave/brave-browser#17146
Fixes brave/brave-browser#17147
Uplift of #9552
Fixes brave/brave-browser#16798
Uplift of #9605
Fixes brave/brave-browser#17138
Fixes brave/brave-browser#17210
Pre-approval checklist:
Pre-merge checklist:
Post-merge checklist: