Skip to content

✨ Add global id types#51

Merged
RobertJCarroll merged 2 commits into
mainfrom
id_types
Jun 15, 2026
Merged

✨ Add global id types#51
RobertJCarroll merged 2 commits into
mainfrom
id_types

Conversation

@RobertJCarroll

Copy link
Copy Markdown
Collaborator

This is an implementation of defining custom types (for pt and ob global id types) to test a possible global id specification implementation.

In the derived SQL, they all just appear as text types.

@github-actions

github-actions Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://include-dcc.github.io/common-access-model/pr-preview/pr-51/

Built to branch gh-pages at 2026-06-12 20:19 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Increases the number of leveraged global id types.
@RobertJCarroll

Copy link
Copy Markdown
Collaborator Author

I added some additional global id types in. A couple of gaps still:

  1. We are missing a few global ID types in dewrangle: FamilyMemberHistory, PlanDefinition, and MedicationStatement.
  2. It may be sensible to rearrange the specimen related items such that there are not ids without a global id match. Specifically, putting them in Sample with inlined=F may be appropriate here.

@RobertJCarroll RobertJCarroll merged commit 0e3946d into main Jun 15, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant