-
Notifications
You must be signed in to change notification settings - Fork 893
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
New onboarding changes android #26999
Conversation
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.
lgtm
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.
Do we also need changes an about_flags.cc
, as in f49e3df#diff-cacbc437964cc20aaaf741b80ecbea5cfce9ca9e67725476f73f14cbab2269fb ?
android/java/org/chromium/chrome/browser/firstrun/WelcomeOnboardingActivity.java
Outdated
Show resolved
Hide resolved
@mherrmann i don't think we need to expose the flag for users as we can control it through griffin. |
237008d
to
447a69b
Compare
447a69b
to
c814ac8
Compare
c814ac8
to
277eba6
Compare
277eba6
to
333abe0
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.
lgtm
New onboarding changes android
New onboarding changes android
Released in v1.75.110 |
New onboarding changes android Resolve rebase issue
Verification PASSED on
screen-20241218-204843.mp4Also ensured the following so we don't run into issues like brave/brave-browser#42113 & brave/brave-browser#42120:
Ensured that once screen-20241218-214135.mp4 |
Merge pull request #26999 from brave/new_onboarding_changes_android New onboarding changes android Resolve rebase issue
Merge pull request #26999 from brave/new_onboarding_changes_android New onboarding changes android
Resolves brave/brave-browser#42863
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
adb shell "echo ' --enable-features=NewAndroidOnboarding' > /data/local/tmp/brave-test-day-zero-expt"
VIDEO coming soon