-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Copy CCC Global database changes - Version 6 #2815
Conversation
Added table to global DB for storing ConnectKeyRecords (changed DB version to 6).
@damagatchi retest this please |
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.
Code Looks ok to me and will let @OrangeAndGreen confirm if it merges all DB changes from Connect.
Also we will need to test the update from current Playstore version to 2.54 after merging this.
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.
I think we'll need to include the upgrade to V7 as well, which adds a column to the table for ConnectKeyRecord. Users on the latest beta already have this additional change.
Yes, I'm going to raise a separate PR for V7. |
@damagatchi retest this please |
3 similar comments
@damagatchi retest this please |
@damagatchi retest this please |
@damagatchi retest this please |
@shubham1g5 just an update to the externalizable classes list, can I have your approval again? |
Summary
This PR is to copy Global DB changes that were done as part of the CCC work. The goal with bringing these changes is to ensure a healthy update from that version to production. Changes included here are:
Safety Assurance
Safety story
I believe this is safe as we are only adding a new table to the Global DB that is used by CCC.
cross-request: dimagi/commcare-core#1416