-
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
Spi migrate users ups into test spi #1364
Conversation
…e script. Should work...maybe. Needs to be tested and debugged before merge
… lookup names once migation confirms final table values
… opposed to method lookups branch
…o, with period id and visit id
…sons to determine if they have email addresses and give the collab role based on this. Joined on names. Not tested in current state
…name and lastname else default. Untested. May also be a bad idea, needs a review of logic
…_user_id is null and will then be inserted.
… biohub_user_id as system_user_id
Openshift URLs for the PR Deployment: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## test-spi #1364 +/- ##
============================================
+ Coverage 47.88% 50.10% +2.21%
============================================
Files 736 736
Lines 20015 20015
Branches 3146 3146
============================================
+ Hits 9585 10028 +443
+ Misses 9825 9819 -6
+ Partials 605 168 -437 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Links to Jira Tickets
Description of Changes
Testing Notes