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

[97] Fixes schemeURI is never generated in the XML payload #98

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

t-book
Copy link
Contributor

@t-book t-book commented Feb 2, 2024

This PR fixes the missing schemeURI attribute in datacite XML as explained in the issue #97

@alycejenni alycejenni changed the base branch from main to dev February 8, 2024 09:37
@alycejenni alycejenni merged commit 4b3be81 into NaturalHistoryMuseum:dev Feb 8, 2024
@alycejenni
Copy link
Member

Hi @t-book! Thanks for this PR, it's really appreciated.

@t-book
Copy link
Contributor Author

t-book commented Feb 16, 2024

@alycejenni thanks for accepting ! more to come ;)

@alycejenni
Copy link
Member

@t-book if you're planning to make more contributions (which are definitely welcome, thank you!), could I ask that you have a quick read of our guidelines for commits and PRs first?

We know we need to make these a little more obvious to new contributors, but I thought I'd point them out if you're planning to make some more changes!

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

Successfully merging this pull request may close these issues.

2 participants