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

Replace cogatlas prefix for assessment tools #835

Closed
2 tasks
surchs opened this issue Dec 16, 2024 · 0 comments · Fixed by #836
Closed
2 tasks

Replace cogatlas prefix for assessment tools #835

surchs opened this issue Dec 16, 2024 · 0 comments · Fixed by #836
Assignees

Comments

@surchs
Copy link
Contributor

surchs commented Dec 16, 2024

Once we move assessments to SNOMED-CT, we need to also update all hard-coded references to CogAtlas. E.g. here:

toolTerms: Object.entries(toolTerms).map(([key, value]) => ({
label: value,
identifier: "cogatlas:" + key,
selected: false
})),

TODO

  • Update all tests to use snomed over cogatlas
  • Replace CogAtlas prefix with snomed in the codebase
@surchs surchs moved this to Backlog in Neurobagel Dec 16, 2024
@surchs surchs mentioned this issue Dec 16, 2024
2 tasks
@surchs surchs moved this from Backlog to Specify - Active in Neurobagel Dec 17, 2024
@rmanaem rmanaem moved this from Specify - Active to Implement - Active in Neurobagel Dec 17, 2024
@rmanaem rmanaem assigned rmanaem and surchs and unassigned rmanaem Dec 17, 2024
@surchs surchs moved this from Implement - Active to Implement - Done in Neurobagel Dec 18, 2024
@rmanaem rmanaem moved this from Implement - Done to Review - Active in Neurobagel Dec 18, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review - Done
Development

Successfully merging a pull request may close this issue.

2 participants