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

Update glossary.ttl #38

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Update glossary.ttl #38

merged 1 commit into from
Feb 5, 2025

Conversation

leskneebone
Copy link

Fixes implemented as per comments in idn-au/.github#110

Fixes implemented as per comments in idn-au/.github#110
@leskneebone leskneebone requested a review from lalewis1 February 4, 2025 00:47
@leskneebone
Copy link
Author

@lawson after this is updated please confirm the location of this in dev.demo server. The link that is currently logged at idn-au/.github#108 is dead.

Copy link
Collaborator

@lalewis1 lalewis1 left a comment

Choose a reason for hiding this comment

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

lgtm

@lalewis1 lalewis1 merged commit e504b86 into leskneebone-patch-1 Feb 5, 2025
1 check passed
@lalewis1 lalewis1 deleted the leskneebone-patch-3 branch February 5, 2025 03:38
@lalewis1
Copy link
Collaborator

lalewis1 commented Feb 5, 2025

just confirming that you are aware this was a merge request from the leskneebone-patch-3 branch into the leskneebone-patch-1 branch.

did you also want to merge this back to the main branch ?

@leskneebone
Copy link
Author

leskneebone commented Feb 5, 2025

Sorry Lawson I'm a bit lost in the branches now and not sure where things are at.

for testing, what goes back to the main branch should include this triple:

https://data.idnau.org/pid/glossary/63da5279-8772-4f9c-8e55-6f5f6f85129a
https://schema.org/citation
https://doi.org/10.1006/knac.1993.1008

I can't see this triple in patch 1.

@lalewis1
Copy link
Collaborator

lalewis1 commented Feb 6, 2025

yes I can't see it either. but it looks like it was never in patch-1 or patch-3.

maybe you have some local changes that have not been pushed?

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