You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
One of the lessons learned from the Data Source Identification Repo is that a local database would enable ue to test database changes more simply, quickly and reliably compared to a standalone sandbox database.
This would also considerably simplify our database migration logic, as we could eliminate having to account for separate sandbox database logic.
In a pinch, we may also be able to eliminate stage, but that's much less certain
Additionally, this may help with #584 , if we wish to go in that direction.
Steps would be as follows:
Set up a local_database repository within the repo which can be used to spin up a local postgres database
Local environment variables would then need modified to point to the local database (as opposed to a sandbox database)
Set up a README that instructs users to do exactly that
Set up logic such that the SQLAlchemy model is used to construct the database structure
Run existing tests to ensure they still perform as expected in the new environment
If tests fail, that indicates the SQLAlchemy model needs modified to properly reflect the sandbox database -- only models.py should be modified for these to pass.
The Github Runner could then also be modified to use a local database for tests, much like how it's currently done in the data source identification app
The text was updated successfully, but these errors were encountered:
One of the lessons learned from the Data Source Identification Repo is that a local database would enable ue to test database changes more simply, quickly and reliably compared to a standalone sandbox database.
This would also considerably simplify our database migration logic, as we could eliminate having to account for separate sandbox database logic.
Additionally, this may help with #584 , if we wish to go in that direction.
Steps would be as follows:
local_database
repository within the repo which can be used to spin up a local postgres databasemodels.py
should be modified for these to pass.The text was updated successfully, but these errors were encountered: