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

pkp/pkp-lib#4787 Reviewer suggestions #1835

Closed
wants to merge 6 commits into from

Conversation

touhidurabir
Copy link
Member

Copy link
Contributor

@defstat defstat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @touhidurabir, no comments here. Just a common comment that you will find on OJS.

@touhidurabir touhidurabir force-pushed the i4787_main branch 4 times, most recently from 62462f9 to b3f0129 Compare February 25, 2025 11:13
Copy link
Contributor

@defstat defstat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a wording comment here, regarding a previous fix on OJS that may have been forgotten for OMP


msgid "default.submission.step.reviewerSuggestions"
msgstr ""
"<p>When submitting your article, you have the option to suggest 2/3 potential "
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here the wording is different than the one in OJS. Is this intentional?

Copy link
Member Author

@touhidurabir touhidurabir Feb 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right , I missed that . Also wondering if we should centralise in pkp-lib which seems better . One question , instead of article which is OJS specific , what can be used as common word for both OJS and OMP ? like instead of When submitting your article should we say When submitting your submission ? I am not sure about the common term .

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

submitting your submission doesn’t sound quite right to me, but I’m not the best person to determine the most suitable term or wording.

For now, I suggest keeping it context-specific: using article for OJS and monograph for OMP. We can revisit this later if a more unified term becomes necessary.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we don't need to fork locale keys into 3 places, it's much better :)

I think `When submitting, you have the option..." is just as clear and doesn't require app-specific terminology.

@touhidurabir touhidurabir force-pushed the i4787_main branch 2 times, most recently from 26bd4bc to 6a429b7 Compare February 26, 2025 14:00
@asmecher
Copy link
Member

Manually merged to resolve submodule conflict.

@asmecher asmecher closed this Feb 26, 2025
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.

3 participants