-
Notifications
You must be signed in to change notification settings - Fork 884
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
Bookmarks model not loaded (uplift to 1.5.x) #4687
Conversation
CI failed for iOS
for Linux, could not pass phase of init
for macOS, build
|
QA we'll need to run through #4588 (comment) on Nightly before we can approve this into @AlexeyBarabash did you check this on Nightly? |
@kjozwiak The test plan requires custom-built installer which has an introduced delay, so it is impossible to test by those steps by QA, so brave/brave-browser#8178 has I went through steps at #4588 (comment) with own built master (v1.7.5) with a mentioned modification and verified it passes. |
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.5.x
approved after deliberating with @brave/uplift-approvers. Looks like both CI/Jenkins
and CI/Travis
passed without any failures. As per #4688, this will be difficult for QA to check. However, it does appear like we have tests to cover this scenario.
Please make sure that all the correct labels are being used and the associated issue is moved into the correct milestone.
Uplift of #4588
Approved, please ensure that before merging:
After you merge: