-
Notifications
You must be signed in to change notification settings - Fork 3
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: Identify datasets by their URI #1426
Conversation
b2ffd74
to
a44e1a7
Compare
BREAKING CHANGE: terminology sources are no longer identifier by their distribution URI. Keep BC by still allowing the distribution URI to be used when searching. However, the sources query now returns source instead of distribution URIs.
8522429
to
17a88a6
Compare
* Consistently identify variants with hash (#) instead of slash (/). BREAKING CHANGE: terminology sources are no longer identifier by their distribution URI. Keep BC by still allowing the distribution URI to be used when searching. However, the sources query now returns source instead of distribution URIs.
17a88a6
to
40281e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me but we should ask the PO (Ruben) for explicite approval as it a quite a fundamental change.
As long as we continue both distribution and source ID for the time being it looks good to me as well. In the longer run we should communicate the move to dataset ID's first before phasing out distribution ID's. Just to be sure, this PR does not result in a breaking change, right? |
There’s no breaking change for clients when sending queries as they can continue to use the distribution URI as before. However, there may be a breaking change for clients when reading the results, if they want to match the URIs from results with multiple terminology sources with the distribution URIs they used when sending the query. However, the results also contain each source’s name, so I’m hoping that’s not a serious break. |
e1e8dbf
to
290886f
Compare
🎉 This PR is included in version 5.4.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 9.16.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
Fix #1423
Terminology sources are no longer identifier by their distribution URI.
Keep BC by still allowing the distribution URI to be used when searching.
The sources query now returns the new source instead of distribution URIs.