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

FHIR-40567 renaming MolSeq.formatted to MolSeq.file #3061

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

khalifa-aly
Copy link
Contributor

Based on FHIR-40567 and Clinical Genomics IM discussion, it was decided to rename MolSeq.formmated to MolSeq.file.

HL7 FHIR Pull Request

Note: No pull requests will be accepted against ./source unless logged in the HL7 Jira issue tracker.

If you made changes to any files within ./source please indicate the Jira tracker number this pull request is associated with:

Description

Based on FHIR-40567, MolSeq.formatted element was renamed to MolSeq.file.

Based on FHIR-40567 and Clinical Genomics IM discussion, it was decided to rename MolSeq.formmated to MolSeq.file.
@khalifa-aly khalifa-aly requested a review from sirkyp April 7, 2024 18:41
Copy link
Contributor

@sirkyp sirkyp left a comment

Choose a reason for hiding this comment

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

@khalifa-aly - did we have examples that use this attribute and also need to be changed?

@sirkyp
Copy link
Contributor

sirkyp commented Apr 8, 2024

@khalifa-aly - I think we have documentation that references the attribute name as well?

updating the example to reflect the new name of MolSeq.file instead of MolecularSequence.formatted
updating the introduction section to reflect the new name of MolSeq.file instead of MolecularSequence.formatted
updating the notes section to reflect the new name of MolSeq.file instead of MolecularSequence.formatted
@khalifa-aly
Copy link
Contributor Author

I made changes to the documentation, and only one example has MolSeq.file, and changed it. I'll check the build one more time.

@khalifa-aly
Copy link
Contributor Author

@sirkyp thanks for highlighting this. Would you please check the new edits?

@sirkyp
Copy link
Contributor

sirkyp commented May 13, 2024

Looks good to me @khalifa-aly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants