-
Notifications
You must be signed in to change notification settings - Fork 16
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
Share the article's DOI instead of the OJS link. #37
Comments
Hi @thiagolepidus , that sounds good to me. You are very welcome to contribute. Thanks, |
Great, @ronste! Do you have any preference among the suggested implementation alternatives? |
Ohh, sorry. Forgot the alternatives. In the spirit of slowly adpating users to new features I would, for now, go with option 2). Once this is accepted we can make it the default, i.e. go for option 3). Apart from that we have to consider that not all shareable object might have a DOI. In that case the Url should be sent. |
thanks for the PR. I will test it on my system, prepare a relase package and update the plugin gallery. |
We had a request to share the DOI link instead of the OJS link, which has a higher probability of being changed over time. Integrating this functionality into the Shariff plugin seemed ideal to us. Can we submit a pull request with this feature?
We are considering three alternatives:
The text was updated successfully, but these errors were encountered: