Skip to content

Add WoS Reviewer Locator plugin for OJS 3.1 - 3.3#390

Merged
asmecher merged 2 commits into
pkp:mainfrom
Alex-nz:main
Jul 29, 2025
Merged

Add WoS Reviewer Locator plugin for OJS 3.1 - 3.3#390
asmecher merged 2 commits into
pkp:mainfrom
Alex-nz:main

Conversation

@Alex-nz

@Alex-nz Alex-nz commented Jul 16, 2025

Copy link
Copy Markdown

Add Web of Science Reviewer Locator plugin for OJS 3.1 and 3.2 into the plugin gallery.

@Alex-nz

Alex-nz commented Jul 21, 2025

Copy link
Copy Markdown
Author

Hi PKP team, can someone please review and merge this PR.
Thank you

@Alex-nz Alex-nz changed the title Add WoS Reviewer Locator plugin for OJS 3.1 and 3.2 Add WoS Reviewer Locator plugin for OJS 3.1 - 3.3 Jul 29, 2025
@Alex-nz

Alex-nz commented Jul 29, 2025

Copy link
Copy Markdown
Author

I don't understand why md5sum check is failing. No change was made to the file since the previous commit when it succeeded. I've checked the md5sum of the file again and it appears to match the expected. @bozana do you know how to re-run the checks?

@kaitlinnewson

Copy link
Copy Markdown
Member

Hi @Alex-nz, it looks like the tests are passing now, and I've checked the md5s manually as well - sometimes the tests can have issues when there are multiple releases in a single PR. @asmecher ready for your review!

@asmecher asmecher merged commit adaf3db into pkp:main Jul 29, 2025
1 of 2 checks passed
@asmecher

Copy link
Copy Markdown
Member

@Alex-nz, I had merged this, but ended up reverting it -- it looks like you're re-using the same version numbers for new releases. For example, there's already a 1.3.3.0 release in the Plugin Gallery. This makes it hard to debug, and will probably cause problems with changing URLs and checksums. I'd suggest bumping the build number and re-submitting this with new version numbers that haven't been used before.

@bozana

bozana commented Jul 30, 2025

Copy link
Copy Markdown

Hi @asmecher, I think this is a fully new plugin, that does not exist in the plugin gallery yet...

@asmecher

Copy link
Copy Markdown
Member

@Alex-nz and @bozana, ah, my mistake! I misread this as a new revision of the plugin above it in the Plugin Gallery. I've re-added the commits.

@Alex-nz

Alex-nz commented Jul 30, 2025

Copy link
Copy Markdown
Author

Thanks @asmecher and @bozana . That's right, this is a completely new plugin, it just follows the same version number convention as the other one from us.

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.

4 participants