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

Fixed authorization request for claiming DOI on related profiles #4046

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

litvinovg
Copy link
Collaborator

VIVO GitHub issue

  • Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this pull request do?

Fixes authorization request for claiming DOIs on related profiles

How should this be tested?

  • Reproduce the issue
  • Test that the pull request fixes the issue

Interested parties

@VIVO-project/vivo-committers

Candidates for reviewing this PR should have some of the following expertises:

  1. Java

Reviewers' report template

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

@litvinovg litvinovg linked an issue Jan 31, 2025 that may be closed by this pull request
Copy link
Contributor

@matthiasluehr matthiasluehr left a comment

Choose a reason for hiding this comment

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

Works. Profile editors are now able to use the claimer in assigned profiles.

Copy link
Contributor

@chenejac chenejac left a comment

Choose a reason for hiding this comment

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

I have also tested the PR and it is fixing linked issue. I don't have any suggestion for code improvements.

@brianjlowe brianjlowe merged commit a6f8546 into vivo-project:main Feb 10, 2025
4 checks passed
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.

Profile editors aren't able to claim publications for a person.
4 participants