Skip to content

Update country to ods term#252

Merged
samleeflang merged 2 commits intomasterfrom
feature/update-country-vc
Mar 31, 2026
Merged

Update country to ods term#252
samleeflang merged 2 commits intomasterfrom
feature/update-country-vc

Conversation

@samleeflang
Copy link
Copy Markdown
Contributor

Decided not to reuse the ltc term but mind a ods term.
The ltc:hasGeographicalContext has a different meaning and is aimed specifically at the geographical context of the specimen in the VirtualCollection.
The ods term shows for which specific country the VC is significant for, the collecting country of the specimen can be different.

@samleeflang samleeflang requested a review from southeo March 31, 2026 09:56
Copy link
Copy Markdown
Contributor

@southeo southeo left a comment

Choose a reason for hiding this comment

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

sry :(

"$ref": "https://schemas.dissco.tech/schemas/fdo-type/shared-model/0.4.0/target-digital-object-filter.json"
},
"ltc:hasGeographicContext": {
"ods:hasSignificanceForCountries": {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If this is just an array of primatives it should be
ods:significanceForCountries
has -> implies object

Copy link
Copy Markdown
Contributor Author

@samleeflang samleeflang Mar 31, 2026

Choose a reason for hiding this comment

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

So sorry, that is on me. Doing too many things at the same time (and then wasting yours 😞 )

@samleeflang samleeflang requested a review from southeo March 31, 2026 13:37
Copy link
Copy Markdown
Contributor

@southeo southeo left a comment

Choose a reason for hiding this comment

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

🗺️

@samleeflang samleeflang merged commit 02e6d01 into master Mar 31, 2026
1 check 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.

2 participants