Skip to content

Conversation

@taslangraham
Copy link
Contributor

@taslangraham taslangraham changed the title pkp/pkp-lib#11125 Rename email variable during installation pkp/pkp-lib#11125 Fix: Copyediting Stage, Discussion tags do not populate [main] Mar 25, 2025
@taslangraham taslangraham marked this pull request as ready for review March 28, 2025 03:23
@taslangraham taslangraham force-pushed the i11125-main branch 3 times, most recently from 4f32c8f to cb02f30 Compare May 27, 2025 20:17
@taslangraham taslangraham requested a review from ewhanson May 27, 2025 21:27
Copy link
Contributor

@ewhanson ewhanson left a comment

Choose a reason for hiding this comment

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

Same comment about blank lines between locale entries as in stable-3_5_0. Otherwise looks good.


msgid "notification.type.editorAssignmentTask"
msgstr ""

Copy link
Contributor

Choose a reason for hiding this comment

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

One blank line needed between the entries for all locale files.

@taslangraham taslangraham force-pushed the i11125-main branch 2 times, most recently from c67adbe to 6a0ee00 Compare May 28, 2025 18:32
@ewhanson
Copy link
Contributor

@taslangraham, could you squash the "code refactor" commit into one of the others and double check the failing tests? It's not consistent but does seem to be happening across the applications/branches with the upgrade tests.

@ewhanson ewhanson merged commit 35ac50c into pkp:main May 29, 2025
8 checks passed
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.

2 participants