Clinical-Genomics / trailblazer

Keep track of and manage analyses
MIT License
5 stars 2 forks source link

Add user abbreviations to the database #457

Open islean opened 1 week ago

islean commented 1 week ago

Description

This PR attempts to remedy https://github.com/Clinical-Genomics/cigrid-ui/issues/600 although a PR in cigrid-ui will be needed as well as a script. Essentially this PR adds a field to the User table to store the abbreviation of the CG employees to visualize in the FE

Added

How to prepare for test

How to test

Expected test outcome

Review

This version is a

islean commented 1 week ago

Alembic upgrade works well and adds the following column to the User table:

image

Downgrade also works.

sonarcloud[bot] commented 2 days ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

islean commented 2 days ago

Spoke with KSV and the current suggesting is to add a flag for setting it in trailblazer add user and let updates be a manual thing since the goal is for the abbreviations to be constant.