Skip to content
This repository has been archived by the owner on Sep 19, 2018. It is now read-only.

Update Local IndexedDB Schema #88

Merged
merged 3 commits into from
Feb 15, 2018
Merged

Update Local IndexedDB Schema #88

merged 3 commits into from
Feb 15, 2018

Conversation

linuxwolf
Copy link
Contributor

@linuxwolf linuxwolf commented Feb 13, 2018

Resolves #78

Implements the required changes, plus a couple of placeholders. The remainder is safer to accomplish when the tables start to be used.

@ghost ghost assigned linuxwolf Feb 13, 2018
@ghost ghost added the in progress We are actively working on it. label Feb 13, 2018
@m8ttyB
Copy link
Contributor

m8ttyB commented Feb 13, 2018

Do you need a PI review of this work?

@linuxwolf
Copy link
Contributor Author

linuxwolf commented Feb 13, 2018

@m8ttyB I suspect this will be like PR #86, where extra eyes will be welcome but not sure what independent verification is possible ... when it's ready. Still a Work In Progress for now.

@linuxwolf linuxwolf changed the title (WIP) Update Local IndexedDB Schema Update Local IndexedDB Schema Feb 15, 2018
@linuxwolf linuxwolf requested a review from a team February 15, 2018 20:13
@linuxwolf
Copy link
Contributor Author

@mozilla-lockbox/product-integrity should be ready for a once-over.

Copy link

@kimberlythegeek kimberlythegeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

@linuxwolf linuxwolf merged commit 17146cb into mozilla-lockwise:master Feb 15, 2018
@ghost ghost removed the in progress We are actively working on it. label Feb 15, 2018
@linuxwolf linuxwolf deleted the 78-schema branch February 15, 2018 20:58
@m8ttyB
Copy link
Contributor

m8ttyB commented Feb 20, 2018

I see @kimberlythegeek r+ this for PI, adding the pi-verified label.

@kimberlythegeek
Copy link

@m8ttyB ah, yes, sorry

linuxwolf added a commit that referenced this pull request May 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants