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

Remove SFS specific views #96

Merged
merged 3 commits into from
Jan 15, 2020
Merged

Remove SFS specific views #96

merged 3 commits into from
Jan 15, 2020

Conversation

joverlee521
Copy link
Contributor

Removes the following SFS specific views from the core shipping views:

  • incidence_model_observation_v1
  • incidence_model_observation_v2
  • incidence_model_observation_v3
  • observation_with_presence_absence_result_v1
  • observation_with_presence_absence_result_v2

This will need to be deployed with the sqitch --log-only option since there are other views within ID3C customizations that depend on these views and we don't actually want to drop these views.

@joverlee521
Copy link
Contributor Author

These SFS specific shipping views will be ported over to
ID3C-customizations.

This schema change should be deployed with the sqitch's `--log-only`
option since we don't actually want to drop these views from the
database.
Remove the `drop view...` statements used for removing SFS specific
from ID3C core. We do not want to include these statements in future
shipping view reworks so that the SFS views are not dropped from
the database.
Copy link
Member

@tsibley tsibley left a comment

Choose a reason for hiding this comment

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

Repushed with tags renamed so they're not weirdly out of order. The out of order names wouldn't cause issues, other than maybe making our brains hurt. (And they wouldn't have been out of order if I hadn't pushed stuff this morning.)

@joverlee521 joverlee521 merged commit 492638e into master Jan 15, 2020
@joverlee521 joverlee521 deleted the remove-sfs-views branch January 15, 2020 20:14
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