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

Retire the Resolver plugin #10964

Closed
kaitlinnewson opened this issue Feb 21, 2025 · 4 comments · Fixed by pkp/ojs#4690
Closed

Retire the Resolver plugin #10964

kaitlinnewson opened this issue Feb 21, 2025 · 4 comments · Fixed by pkp/ojs#4690
Assignees
Milestone

Comments

@kaitlinnewson
Copy link
Member

kaitlinnewson commented Feb 21, 2025

The Resolver plugin in OJS creates resolvable URLs to publications. With the prominence of PIDs like DOIs however, the plugin seems less likely to be used and could be retired.

A web search for the URL format that the plugin generates returns very few results, suggesting that these URLs are not being used frequently.

What application are you using?
OJS (this plugin is only for OJS)

Additional information

Pull requests

@asmecher
Copy link
Member

It would also be easy to restore if there is a demand for it, but I agree, I suspect it's not used. I suggest removing it to confirm :)

@kaitlinnewson
Copy link
Member Author

@asmecher Would this be appropriate for 3.5 so we can add a migration to remove it from the database - or defer that for later?

@asmecher
Copy link
Member

asmecher commented Feb 25, 2025

Hmm... I would lean towards no, I think. OJS should behave just fine with an extra entry in versions for a gateway plugin that isn't present, and as a general practice plugin data should be able to sit dormant in an installation in case it's useful again later.

Clarification: That's a no to the migration, but a yes to removing it from the github repo for 3.5.0.

@kaitlinnewson kaitlinnewson self-assigned this Feb 25, 2025
@kaitlinnewson kaitlinnewson added this to the 3.5.0 LTS milestone Feb 25, 2025
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Feb 25, 2025
@kaitlinnewson kaitlinnewson linked a pull request Feb 25, 2025 that will close this issue
asmecher pushed a commit to pkp/ojs that referenced this issue Feb 26, 2025
@asmecher
Copy link
Member

Merged, thanks!

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 a pull request may close this issue.

2 participants