-
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
USSBookmarks migration and sync fixes (uplift to 1.4.x) #4412
Conversation
Restarting Restarting |
Looks like it failed to fetch the repo on |
per #4324 (comment), let's hold off on uplift. Will mark as blocked |
Still |
Sync bookmarks ordering fix
Restarting Restarting |
Restarting one last time as the same failures occurred as per #4412 (comment) |
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.4.x
approved after deliberating with @brave/uplift-approvers. Please ensure that all the correct labels are being used and the associated issue(s) are merged into the correct milestone.
@btlechowski also ran rough the PRs on Nightly as per #4324 (comment).
Uplift of #4324 (USSBookmarks migration)
Fixes brave/brave-browser#6437
Uplift of #4420 (Ignore bookmark icon url when browser gets bookmark form sync cloud)
Fixes brave/brave-browser#7893
Uplift of #4458
Fixes brave/brave-browser#7948
Approved, please ensure that before merging:
After you merge: