-
Notifications
You must be signed in to change notification settings - Fork 9
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
SIMSBIOHUB-148: Create New Agency Database Table to Support All Agencies #1057
Conversation
- remove static coordinator agency list - update coordinator agency codes list to use funding_source values
Update readme to include sample migration code and naming conventions, etc.
Openshift URLs for the PR Deployment: |
Codecov Report
@@ Coverage Diff @@
## dev #1057 +/- ##
==========================================
- Coverage 67.76% 67.73% -0.03%
==========================================
Files 403 403
Lines 12481 12473 -8
Branches 2101 2101
==========================================
- Hits 8458 8449 -9
Misses 3489 3489
- Partials 534 535 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Openshift URLs for the PR Deployment: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ran locally and things are working as expected. Great examples in the read me
database/src/migrations/20230719000000_update_funding_sources.ts
Outdated
Show resolved
Hide resolved
ef5e3e3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🦖
SonarCloud Quality Gate failed. 0 Bugs No Coverage information Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran locally, works as intended! 😹
Links to Jira Tickets
https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-148
Description of Changes
Funding Source -> Agency
funding_source
table toagency
.funding_source
in their name. Rename to useagency
.tr_project_funding_source
trigger to reference renamedagency
table and columns.funding_source
to useagency
fs
(aka:funding_source
) toagency
pfs
toproject_funding_source
for clarityOther
Documentation
Testing Notes