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.

Sign up for GitHub

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

Jump to bottom

Expose the reviewed preprint's uuid in the /rapidReviews endpoint #225

Closed
harumhelmy opened this issue Mar 3, 2021 · 2 comments
Closed

Expose the reviewed preprint's uuid in the /rapidReviews endpoint #225

harumhelmy opened this issue Mar 3, 2021 · 2 comments
Assignees
@harumhelmy
Labels
API Relates to the API specification, design, and documentation backend Involves the site backend enhancement New feature or request quick fix This issue is an easy/fast fix

Comments

@harumhelmy
Copy link
Contributor

harumhelmy commented Mar 3, 2021 •
edited
Loading

Per this issue in the former OSrPRE repo, bioRxiv would like the following information to be exposed in order to be able to display/link to PREreview content on their preprint pages:

  • preprint DOI,
  • review title,
  • date,
  • reviewer author name,
  • link back to PREreview

Our API exposes all the above yay 🎉 except that the /rapidReviews GET index endpoint doesn't expose the uuid of the reviewed preprint, which bioRxiv will need to link back to PREreview. It's be an easy fix, but I just want to make sure I'm not overlooking anything, @jheretic
@harumhelmy harumhelmy added enhancement New feature or request backend Involves the site backend quick fix This issue is an easy/fast fix API Relates to the API specification, design, and documentation labels Mar 3, 2021
@harumhelmy harumhelmy self-assigned this Mar 3, 2021
@jheretic
Copy link
Contributor

jheretic commented Mar 3, 2021

That sounds right. I'm working on some additional API fixes on the api-permissions branch, I'll just slot this change in there since I'm aiming that for today anyway, if that sounds good to you @harumhelmy

@murkatr
Copy link
Contributor

murkatr commented Mar 4, 2021

@harumhelmy @jheretic I believe for preprints with multiple rapid and long-form reviews, we also wanted to provide the option of displaying the count of reviews instead of listing all the reviews individually. @dasaderi does this sound correct?

@jheretic jheretic mentioned this issue Mar 8, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@harumhelmy harumhelmy

Labels
API Relates to the API specification, design, and documentation backend Involves the site backend enhancement New feature or request quick fix This issue is an easy/fast fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jheretic @harumhelmy @murkatr