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

Support Schema version 14 #146

Merged
merged 2 commits into from
Aug 13, 2023
Merged

Support Schema version 14 #146

merged 2 commits into from
Aug 13, 2023

Conversation

AndreasSko
Copy link
Owner

With the latest JW Library update the schema version of the manifest has increased to 14 now. The sqldiff didn't show
anything special:

sqldiff /Users/askorczyk/src/GitHub/go-jwlm/model/data/userData.db /Users/askorczyk/Downloads/UserdataBackup_2023-08-13_iPhone/userData.db
UPDATE LastModified SET LastModified='2023-08-13T10:07:57Z' WHERE rowid=1;
UPDATE grdb_migrations SET identifier='v14' WHERE rowid=1;

So it should be safe to simply update the schema version without adjusting the model.

Other changes: More granular error message on manifest validation

Instead of just saying that the version is incompatible, it is communicated if the received version is either too old or too new.

The schema version of the manifest has
increased to 14 now. The sqldiff didn't show
any special:

```
sqldiff /Users/askorczyk/src/GitHub/go-jwlm/model/data/userData.db /Users/askorczyk/Downloads/UserdataBackup_2023-08-13_iPhone/userData.db
UPDATE LastModified SET LastModified='2023-08-13T10:07:57Z' WHERE rowid=1;
UPDATE grdb_migrations SET identifier='v14' WHERE rowid=1;
```

So it should be save to simply update
the schema version without adjusting the
model.
Instead of just saying that the version is
incompatible, it is communicated if the received
version is either too old or too new.
@coveralls
Copy link

Coverage Status

coverage: 89.358% (-0.005%) from 89.363% when pulling cfb9d5a on schema_14 into 3e0a485 on master.

@AndreasSko AndreasSko merged commit f400981 into master Aug 13, 2023
8 checks passed
@AndreasSko AndreasSko deleted the schema_14 branch August 13, 2023 14:19
AndreasSko added a commit to AndreasSko/ios-jwlm that referenced this pull request Aug 13, 2023
With the latest JW Library update the schema
version of the manifest has increased to 14
now.
Also see AndreasSko/go-jwlm#146
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.

2 participants