Skip to content

/ brave-browser

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

Jump to bottom

Update sync DEPS for missing parts to decrypt solution #6675

Closed
darkdh opened this issue Oct 30, 2019 · 0 comments
Closed

Update sync DEPS for missing parts to decrypt solution #6675

darkdh opened this issue Oct 30, 2019 · 0 comments
Assignees
@darkdh
Labels
QA/No feature/sync release-notes/exclude
Projects
Sync - All platforms
Milestone
1.0.0 - Release

Comments

@darkdh
Copy link
Member

@darkdh darkdh commented Oct 30, 2019

Original issue: brave/sync#289
The fix is in brave/sync#350 which includes compaction API but it is ok to pull the DEPS because compaction API need explicitly calling it to use.
@darkdh darkdh self-assigned this Oct 30, 2019
@darkdh darkdh added this to Untriaged backlog in Sync - All platforms via automation Oct 30, 2019
@darkdh darkdh moved this from Untriaged backlog to In Progress in Sync - All platforms Oct 30, 2019
@darkdh darkdh mentioned this issue Oct 30, 2019
Merged
5 of 32 tasks complete
Sync - All platforms automation moved this from In Progress to Completed Oct 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@darkdh darkdh

Labels
QA/No feature/sync release-notes/exclude
Projects
Sync - All platforms
  
Completed
Milestone
  1.0.0 - Release
Linked pull requests

Successfully merging a pull request may close this issue.

Update brave sync DEPS to support cross batch decryption
2 participants
@bsclifton @darkdh
You can’t perform that action at this time.