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

Disable orcid synchronization per Type #84

Open
floriangantner opened this issue Mar 26, 2024 · 0 comments
Open

Disable orcid synchronization per Type #84

floriangantner opened this issue Mar 26, 2024 · 0 comments

Comments

@floriangantner
Copy link

Is your feature request related to a problem? Please describe.
As an repository manager i might not want the user to set their orcid sync settings for projects, patents or products, because my repository does not have that entity configured or does have too poor data the fulfill the minimal requirements for this entity in future (e.g. funding identifier missing)

Describe the solution you'd like
We would like the synchronization settings on the frontend to be shown based on some Authorization Feature checking if the synchronization is currently enabled for the specific entity type, similar to the https://github.com/4Science/DSpace/blob/main-cris/dspace-server-webapp/src/main/java/org/dspace/app/rest/authorization/impl/CanSynchronizeWithORCID.java Feature.

We also would like some addtional check in the OrcidHistoryServiceImpl.synchronizeWithOrcid https://github.com/4Science/DSpace/blob/f768bbe0f1017a198262e7bc3aaa86aa332ba1d2/dspace-api/src/main/java/org/dspace/orcid/service/impl/OrcidHistoryServiceImpl.java#L178 checking if the synchronization is currently enabled for the Type of the OrcidQueue which is being pushed.

Describe alternatives or workarounds you've considered

Current workaround is to comment out the corresponding entity on the orcid sync component the

and comment out occurences of the the form data being processed in https://github.com/4Science/dspace-angular/blob/main-cris/src/app/item-page/orcid-page/orcid-sync-settings/orcid-sync-settings.component.ts .

Additional context

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

No branches or pull requests

1 participant