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

Add more validations for *SnapshotContentModels #8372

Open
vojtechjelinek opened this issue Jan 8, 2020 · 0 comments
Open

Add more validations for *SnapshotContentModels #8372

vojtechjelinek opened this issue Jan 8, 2020 · 0 comments

Comments

@vojtechjelinek
Copy link
Member

Some of the *SnapshotContentModels we have in out datastore are not valid. Some of the snapshot models contain fields that no longer exist in their parent model and that will cause an exception when we try to revert the parent model to some older version.

We should add more checks into the prod validation jobs for the *SnapshotContentModels, these checks would run the populate() method on the parent model a try to populate it with the data that are inside the snapshot in the content field, that way we can find out if we can revert to all of the previous versions.

@vojtechjelinek vojtechjelinek self-assigned this Jan 8, 2020
@vojtechjelinek vojtechjelinek added this to To do in Data handling Jul 3, 2020
@vojtechjelinek vojtechjelinek moved this from TODO (critical) to TODO (improvement/feature) in Data handling Oct 13, 2020
@seanlip seanlip removed this from TODO (improvement/feature) in Data handling Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

2 participants