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

Solr-API-app #532

Merged
merged 18 commits into from
Feb 25, 2019
Merged

Solr-API-app #532

merged 18 commits into from
Feb 25, 2019

Conversation

rarmitag
Copy link
Collaborator

Issue #, if available: - bcgov/entity#24

Completed app:

Synonyms-
added stems_text column for searches

Restricted words -
4th table created to display and edit.

Decision Reason

Audit tables for each editable page

Two databases queried for data

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Copy link
Contributor

@katiemcgoff katiemcgoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove the whole migration framework (I know, this goes against what we set up in November :D). That way if one of us (current and future developers) forgets that we don't do migrations/db changes from this app and tries, it will be harder.

@katiemcgoff katiemcgoff merged commit 5262e89 into bcgov:master Feb 25, 2019
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.

4 participants