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

feat: CQDG-425 map refseq_mrna_id in variant consequences table #172

Closed
wants to merge 1 commit into from

Conversation

atoulous
Copy link
Contributor

feat: CQDG-425 map refseq_mrna_id in variant consequences table

  • closes #TICKET_NUMBER

Description

https://ferlab-crsj.atlassian.net/browse/CQDG-425

Acceptance Criterias

Validation

  • Code Approved
  • QA Done
  • Design/UI Approved from design

Screenshot

image

QA

Steps to validate
Url (storybook, ...)
...

Mention

? refseq_mrna_id.map((id, index) => (
<ExternalLink
key={index}
href={`https://www.ncbi.nlm.nih.gov/nuccore/${refseq_mrna_id}?report=graph`}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je dis peut être une grosse bétise mais c'est pas plutôt id que tu veux mettre dans l'url sachant que refseq_mrna_id est un tableau ?

Suggested change
href={`https://www.ncbi.nlm.nih.gov/nuccore/${refseq_mrna_id}?report=graph`}
href={`https://www.ncbi.nlm.nih.gov/nuccore/${id}?report=graph`}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non on veut les ids combinés ici

@github-actions
Copy link

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 0/0
🟢 Branches 100% 0/0
🟢 Functions 100% 0/0
🟢 Lines 100% 0/0

Test suite run success

0 tests passing in 0 suite.

Report generated by 🧪jest coverage report action from 4f8136c

@atoulous atoulous closed this Oct 25, 2023
@atoulous atoulous deleted the feat/cqdg-425 branch October 25, 2023 14:20
@atoulous atoulous restored the feat/cqdg-425 branch October 25, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants