Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

orcid: change legacy URL #947

Merged
merged 1 commit into from
Sep 28, 2023
Merged

orcid: change legacy URL #947

merged 1 commit into from
Sep 28, 2023

Conversation

jma
Copy link
Contributor

@jma jma commented Sep 18, 2023

ORCID has changed their base URL for oauth. The old URL is a redirect which
is not supported by invenio.

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@jma jma marked this pull request as ready for review September 19, 2023 08:18
@jma jma requested review from mmo and Garfield-fr September 19, 2023 08:18
ORCID has changed their base URL for oauth. The old URL is a redirect which
is not supported by invenio.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@PascalRepond PascalRepond merged commit 60ddf88 into rero:staging Sep 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants