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

Y24-081 Add Program name field to MLWH #4109

Closed
neilsycamore opened this issue May 10, 2024 · 3 comments · Fixed by sanger/unified_warehouse#660, #4265 or #4271
Closed

Y24-081 Add Program name field to MLWH #4109

neilsycamore opened this issue May 10, 2024 · 3 comments · Fixed by sanger/unified_warehouse#660, #4265 or #4271
Assignees
Labels
Enhancement New feature or request

Comments

@neilsycamore
Copy link
Contributor

neilsycamore commented May 10, 2024

As team lead Sequencing Operations (James M) we would like to add the “name” field from the “Programs” table in the Sequencescape database to the MLWH. This will enable us to identify studies by Sanger programme type and enable efficient reporting.

Contact
jm36

Raised in RT802591

@neilsycamore neilsycamore added the Research Research only label May 10, 2024
@SujitDey2022
Copy link

@TWJW-SANGER @stevieing this was marked as low priority in the requirement sheet Seq80 but can be pulled ahead if a quick win.

@sabrine33 sabrine33 self-assigned this Aug 7, 2024
@sabrine33 sabrine33 added Enhancement New feature or request and removed Research Research only labels Aug 8, 2024
@neilsycamore
Copy link
Contributor Author

@sabrine33 the column name should be 'programme' see here https://www.sanger.ac.uk/science/programmes/
The 'program' in Sequencescape is incorrect but no one can see it, if we expose it incorrectly in MLWH I'm sure it will not be long before someone points out the error.

@sabrine33
Copy link
Contributor

@sabrine33 the column name should be 'programme' see here https://www.sanger.ac.uk/science/programmes/ The 'program' in Sequencescape is incorrect but no one can see it, if we expose it incorrectly in MLWH I'm sure it will not be long before someone points out the error.

Thank you @neilsycamore for highlighting this, I will update it accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
3 participants