Skip to content
This repository has been archived by the owner. It is now read-only.

/ browser-laptop Archived

Call prune when synopsis is saved #11482

Closed
NejcZdovc opened this issue Oct 12, 2017 · 0 comments
Closed

Call prune when synopsis is saved #11482

NejcZdovc opened this issue Oct 12, 2017 · 0 comments
Assignees
@NejcZdovc
Labels
QA/no-qa-needed feature/rewards initiative/bat-payments release-notes/exclude
Milestone
0.19.x Hotfix 3 (...

Comments

@NejcZdovc
Copy link
Member

@NejcZdovc NejcZdovc commented Oct 12, 2017

Description

When we save synopsis in user state we need to save it as Immutable.fromJS(synopsis.toJSON()) so that synopsis.prune() is called.
@NejcZdovc NejcZdovc added this to the 0.19.x Release 2 (Beta Channel) milestone Oct 12, 2017
@NejcZdovc NejcZdovc self-assigned this Oct 12, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 30, 2017
@NejcZdovc
Adds prune (clen up) to the ledger code
Unverified
This user has not uploaded their public key yet.
GPG key ID: 584931E990AC6064 Learn about signing commits
58ee537 
Resolves brave#11482

Auditors:

Test Plan:
@NejcZdovc NejcZdovc mentioned this issue Oct 30, 2017
Merged
5 of 8 tasks complete
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 30, 2017
@NejcZdovc
Adds prune (clean up) to the ledger code
Unverified
This user has not uploaded their public key yet.
GPG key ID: 584931E990AC6064 Learn about signing commits
56ea300 
Resolves brave#11482

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 31, 2017
@NejcZdovc
Adds prune (clean up) to the ledger code
Unverified
This user has not uploaded their public key yet.
GPG key ID: 584931E990AC6064 Learn about signing commits
65ff0b7 
Resolves brave#11482

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 31, 2017
@NejcZdovc
Adds prune (clean up) to the ledger code
Unverified
This user has not uploaded their public key yet.
GPG key ID: 584931E990AC6064 Learn about signing commits
87c853e 
Resolves brave#11482

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 31, 2017
@NejcZdovc
Adds prune (clean up) to the ledger code
Unverified
This user has not uploaded their public key yet.
GPG key ID: 584931E990AC6064 Learn about signing commits
1bb075c 
Resolves brave#11482

Auditors:

Test Plan:
bsclifton added a commit that referenced this issue Oct 31, 2017
@bsclifton
Merge pull request #11717 from NejcZdovc/hotfix/#11482-prune
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
9afded3 
Adds prune (clean up) to the ledger code
bsclifton added a commit that referenced this issue Oct 31, 2017
@bsclifton
Merge pull request #11717 from NejcZdovc/hotfix/#11482-prune
Unverified
This user has not uploaded their public key yet.
GPG key ID: 37FF5F19022427F2 Learn about signing commits
Loading status checks…
df4ef66 
Adds prune (clean up) to the ledger code
bsclifton added a commit that referenced this issue Oct 31, 2017
@bsclifton
Merge pull request #11717 from NejcZdovc/hotfix/#11482-prune
Unverified
This user has not uploaded their public key yet.
GPG key ID: 37FF5F19022427F2 Learn about signing commits
Loading status checks…
56ef8c5 
Adds prune (clean up) to the ledger code
bsclifton added a commit that referenced this issue Oct 31, 2017
@bsclifton
Merge pull request #11717 from NejcZdovc/hotfix/#11482-prune
Unverified
This user has not uploaded their public key yet.
GPG key ID: 37FF5F19022427F2 Learn about signing commits
Loading status checks…
39a24ac 
Adds prune (clean up) to the ledger code
syuan100 added a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
@NejcZdovc @syuan100
Adds prune (clean up) to the ledger code
d007cf7 
Resolves brave#11482

Auditors:

Test Plan:
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
QA/no-qa-needed feature/rewards initiative/bat-payments release-notes/exclude
Projects
None yet
Milestone
  0.19.x Hotfix 3 (...
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@bbondy @luixxiul @NejcZdovc
You can’t perform that action at this time.