Skip to content
This repository has been archived by the owner on Nov 12, 2024. It is now read-only.

API version 3 plans? #68

Closed
iainmwallace opened this issue May 31, 2019 · 4 comments · Fixed by #70
Closed

API version 3 plans? #68

iainmwallace opened this issue May 31, 2019 · 4 comments · Fixed by #70
Milestone
v0.5

Comments

@iainmwallace
Copy link

Hi,

I was wondering if there were any plans to support v3 of the API that was recently announced https://orcid.org/blog/2019/05/16/orcid-api-30-here?

One feature that is particularly exciting is the standardization of identifiers.

Thanks

Iain
@sckott
Copy link
Contributor

sckott commented May 31, 2019

thanks for the ping @iainmwallace I didn't know about the new API, will subscribe to the blog so I don't miss these in the future

work on branch https://github.com/ropensci/rorcid/tree/api-v3

@sckott sckott added this to the v0.5 milestone May 31, 2019
sckott added a commit that referenced this issue May 31, 2019
@sckott
#68 added new functions for new API routes in api v3
fba7691 
for those API routes and fxns where theres orcid, orcid with put, and orcid/put/summary, use helper fxn
sckott added a commit that referenced this issue May 31, 2019
@sckott
fix missing summary param for new methods #68
b47b6c8
@sckott
Copy link
Contributor

sckott commented May 31, 2019

@iainmwallace can install remotes::install_github("ropensci/rorcid@api-v3"), reload R, and try it out

@iainmwallace
Copy link
Author

Looks great!

@sckott
Copy link
Contributor

sckott commented Jun 3, 2019

@iainmwallace great, glad it works. will merge soon to master

@sckott sckott mentioned this issue Jun 5, 2019
Merged
@sckott sckott closed this as completed in #70 Jun 5, 2019
sckott added a commit that referenced this issue Jun 5, 2019
@sckott
#68 added new functions for new API routes in api v3
2d86b1d 
for those API routes and fxns where theres orcid, orcid with put, and orcid/put/summary, use helper fxn
sckott added a commit that referenced this issue Jun 5, 2019
@sckott
fix missing summary param for new methods #68
728f9a7
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.5
Development

Successfully merging a pull request may close this issue.

Api v3 done ropensci-archive/rorcid
2 participants
@sckott @iainmwallace