-
Notifications
You must be signed in to change notification settings - Fork 39
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
[MDS-5707] Added history view to Tailings Storage Facilities #3075
Conversation
Quality Gate passed for 'bcgov-sonarcloud_mds_minespace-web'Issues Measures |
Quality Gate failed for 'bcgov-sonarcloud_mds_common'Failed conditions |
Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'Failed conditions |
Quality Gate passed for 'bcgov-sonarcloud_mds_core-web'Issues Measures |
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.
Looks awesome! Left a couple of questions regarding usage of the diff table elements and continuum, but I love it!
* MDS-5708 Updated basic info page * [MDS-5707] Added history view of TSF * MDS-5707 Cleanup of tsf changes * MDS-5707 Cleanup * MDS-5707 Fixed broken tests * Updated TSP for core
Objective
MDS-5707
History View
Added a TSF History view to the basic info page in core/minespace.
SQLAlchemy Continuum
Introduced SQLAlchemy Continuum for "automatic" handling of version tracking. Basically, just add a
__versioned__ = {}
property to your model and it'll create a Version model behind the scene that gets automatically added to on insert, updates, and deletes.Migrations
Added a utility command
make generate_version_table_migration TABLE=<table_name>
that you can use to automatically generate a database migration for the_version
table you want to create, and a DB migration to backfill existing data.