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

Implementation of new data model definitions #289

Merged
merged 11 commits into from
May 14, 2021
Merged

Conversation

howard-e
Copy link
Contributor

@howard-e howard-e commented May 12, 2021

Following the drafting up of a new database schema, this aims to provide new Sequelize models to support that migration.

Note that this introduces breaking changes to how models are referenced on the server, queries and existing import scripts.

NB. This was branched off of entity-tests so certain files may be duplicated here as well as tasks that may have come out of the discussion for that related PR #287.

If your database has not been initialized, run:

if [[ "$OSTYPE" == "darwin"* ]]; then
    createdb # run this if the PostgreSQL installation is freshly installed
    yarn db-init:dev;
else
    sudo -u postgres yarn db-init:dev;
fi;

yarn sequelize db:migrate;

Otherwise, you should be able to safely migrate up with:

yarn sequelize db:migrate

@howard-e howard-e requested review from alflennik and jesdaigle May 12, 2021 15:43
howard-e added 2 commits May 12, 2021 11:54
# Conflicts:
#	server/tests/models/At.spec.js
#	server/tests/models/AtMode.spec.js
#	server/tests/models/AtVersion.spec.js
#	server/tests/models/Browser.spec.js
#	server/tests/models/BrowserVersion.spec.js
@howard-e howard-e merged commit 62315ee into develop May 14, 2021
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