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

ORCID plugin should not function without credentials entered #177

Closed
librariam opened this issue May 20, 2021 · 10 comments
Closed

ORCID plugin should not function without credentials entered #177

librariam opened this issue May 20, 2021 · 10 comments
Assignees

Comments

@librariam
Copy link

Is your feature request related to a problem? Please describe.
Here is the most common scenario and a source of 99% of ORCID-related tickets filed with PKP|PS:

  • A JM/JE enables the plugin without obtaining/entering credentials
  • Authors start receiving ORCID emails with sandbox URL which does not work
  • JM/JE report this as a tech issue with the plugin
  • Following the explanation & instructions on plugin setup most give up on it altogether

Unfortunately I find that this initial experience sours the first impression and turns people away from using the plugin.

Describe the solution you'd like
The bottom line is that the plugin should not be performing any of its functionality (particularly not direct author interactions) without being set up first. This follows the logic of #161

Some ideas for implementation:

  • an additional checkbox within the plugin settings that will not become active until credentials are entered and validated
  • a pop up when enabling the plugin that alerts the JM/JE that it needs to be set up + link to the ORCID guide
  • disable the plugin unless credentials are entered
@asmecher
Copy link
Member

@withanage, this would be a good improvement to include in the next release.

@withanage
Copy link
Member

withanage commented May 22, 2021

@librariam @asmecher

sure, I will add the following settings into the next release.

  • an additional checkbox within the plugin settings that will not become active until credentials are entered and validated
  • disable the plugin unless credentials are entered

@asmecher , is this generally possible currently for adding this kind of features when enabling plugins?

  • a pop up when enabling the plugin that alerts the JM/JE that it needs to be set up + link to the ORCID guide

@asmecher
Copy link
Member

@withanage, just off the top of my head, the PLN plugin had some conditions that were required in order for the plugin to be enabled:
https://github.com/pkp/pln/blob/ojs-stable-3_1_2/PLNPlugin.inc.php#L375

@withanage
Copy link
Member

withanage commented Jul 26, 2021

Hi @librariam ,

I have implemented the plugin as per your definition, before releasing it and doing some more testing, I wanted to ask, if the implementation corresponds to your requirements?

Here is the workflow.

  1. At the beginning plugin is disabled.
  2. Settings is available also when the plugin is disabled.
  3. Status shows about the status of the plugin. (I will link the plugin doc here)
  4. With wrong.-credentials you cannot enable the plugin.
  5. With correct credentials you can enable the plugin.
  6. If globally configured the same rules for client validation apply and the save button is disabled.

One thing, which I could not add was a separate pop-up. But the plugin is not allowed to be enabled as per definition 2.

orcid_validation

@librariam
Copy link
Author

Thank you @withanage . Just to clarify - what would be the behaviour if someone tries to enable the plugin without entering credentials?

@withanage
Copy link
Member

@librariam , users cannot enable the plugin, until they enter credentials (valid) through the backend. They can click the enable plugin, but it will not enable it.

@librariam
Copy link
Author

@withanage thank you, this is great! Not being able to enable the plugin without entering valid credentials is exactly what we're after!

@withanage
Copy link
Member

@librariam

I have released this enhancements for

and integrated in the plugin-gallery.

@asmecher
Copy link
Member

@withanage, is this issue ready to be closed?

@withanage
Copy link
Member

@asmecher There was no bugs reported after adding the functionality in around 2 weeks. Yes, I will close it.

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

3 participants