Skip to content
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

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

avazirna
Copy link
Contributor

@avazirna avazirna commented Aug 16, 2024

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:

  • Addition of table to global DB for storing ConnectKeyRecords (changed DB version to 6)

Safety Assurance

  • I have confidence that this PR will not introduce a regression for the reasons below

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

Added table to global DB for storing ConnectKeyRecords (changed DB version to 6).
@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna avazirna marked this pull request as ready for review August 16, 2024 13:07
@avazirna avazirna self-assigned this Aug 16, 2024
Copy link
Contributor

@shubham1g5 shubham1g5 left a 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.

Copy link
Contributor

@OrangeAndGreen OrangeAndGreen left a 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.

@avazirna
Copy link
Contributor Author

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.

@avazirna
Copy link
Contributor Author

avazirna commented Aug 17, 2024

@damagatchi retest this please

3 similar comments
@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna
Copy link
Contributor Author

@shubham1g5 just an update to the externalizable classes list, can I have your approval again?

@avazirna avazirna merged commit 1d2f00b into commcare_2.54 Aug 19, 2024
1 check passed
@avazirna avazirna deleted the copy-connect-database-upgrades branch August 19, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants