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.

Sign up for GitHub

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

Jump to bottom

Issue559 #560

Merged
merged 7 commits into from
Aug 8, 2024
Merged

Issue559 #560

merged 7 commits into from
Aug 8, 2024

Conversation

smnorris
Copy link
Owner

@smnorris smnorris commented Aug 8, 2024

No description provided.

@smnorris
Copy link
Owner Author

smnorris commented Aug 8, 2024

PRs don't run migrations, this will have to be checked on a _dev db manually.

@smnorris
Copy link
Owner Author

smnorris commented Aug 8, 2024

@NewGraphEnvironment - FYI, new column is modelled_crossing_office_review_date. Let me know if that's no good.

@smnorris
Copy link
Owner Author

smnorris commented Aug 8, 2024

I'll merge, updating the output bcfishpass.crossings_vw to include this new column.

Not sure how long this will be functional though - @nickw-CWF @andrewp-CWF @oakoppel-1 - I can't make informed schema changes without some guidance on how you will be tracking QA.
Is the tracking anticipated to be held in the bcfishpass database?
Will we be able to update QA status in crossings_vw as more data QA is done by CWF?

@smnorris
Merge branch 'main' into issue559
5928263 
* main:
  Added permission assignment to auto_rank.py (#561)
  Updates to rankings (#558)
  LNIC_Updates_2024-07-30 (#557)
  Thom/Shu trib review updates (#554)
@smnorris smnorris merged commit 7d24a49 into main Aug 8, 2024
2 checks passed
smnorris added a commit that referenced this pull request Aug 13, 2024
@smnorris
Merge branch 'main' into ranking-update
075ee64 
* main:
  re-order drops
  drop all fptwg reporting and manually restore after migration
  add missing migrate script
  Issue559 (#560)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@smnorris