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

Add .zenodo.json #478

Merged
merged 7 commits into from
Jan 22, 2025
Merged

Add .zenodo.json #478

merged 7 commits into from
Jan 22, 2025

Conversation

navidcy
Copy link
Collaborator

@navidcy navidcy commented Jan 21, 2025

continuing and closes #472

cc @dougiesquire

@navidcy
Copy link
Collaborator Author

navidcy commented Jan 21, 2025

@dougiesquire could you have a look? Have I done the .json conversion correct?
Make changes/edits to it as needed please!

@navidcy
Copy link
Collaborator Author

navidcy commented Jan 21, 2025

PS: I used the export to JSON option from Zenodo and took the creators part from there. Not sure if that's good idea...

@dougiesquire
Copy link
Collaborator

Thanks @navidcy!

PS: I used the export to JSON option from Zenodo and took the creators part from there. Not sure if that's good idea...

Yeah, there're some strange looking entries in there... I can take a closer look and tidy up tonight

@dougiesquire
Copy link
Collaborator

@navidcy I've tidied things up a little. Let me know what you think.

@max-anu, @micaeljtoliveira, @marc-white, please add your orcid entries if you have them

@navidcy
Copy link
Collaborator Author

navidcy commented Jan 21, 2025

I think is great! I can't approve because I opened the PR but whenever you feel this is ready approve and I'll merge!

@navidcy
Copy link
Collaborator Author

navidcy commented Jan 21, 2025

@micaeljtoliveira is this you? --> https://orcid.org/0000-0003-1364-0907

It does check out with what is mentioned at https://www.access-nri.org.au/person/dr-micael-oliveira/

this is how the Uni name was coming up when I was adding people manually via Zenodo repo using their orcid ids
Copy link
Collaborator

@dougiesquire dougiesquire left a comment

Choose a reason for hiding this comment

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

LGTM

@navidcy navidcy merged commit 24a3551 into main Jan 22, 2025
3 checks passed
@navidcy navidcy deleted the ncc/add-zenodo-json branch January 22, 2025 03:19
@micaeljtoliveira
Copy link
Contributor

@navidcy Yes, that's me! Hmm, looks like I need to update it...

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.

Citation
3 participants