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

[Submission] | Discussion - Copyediting Stage, Discussion tags do not populate #11125

Open
1 task done
mbakioglu opened this issue Mar 18, 2025 · 4 comments
Open
1 task done
Assignees
Labels
Bug:0:Trivial A bug mainly used for UI bugs that do not affect common functionality but break user experience.
Milestone

Comments

@mbakioglu
Copy link

Valid Title

  • I have updated the title to accurately reflect the bug description

Description

Test: Take a new submission through the internal and external review stages
Status: Failed

In the Copyediting Stage, discussion titles and notifications do not populate the tags as expected.

Steps to Reproduce

1.Start a copyediting discussion as a Press Editor to request copyediting from the Copyeditor.
2.In the discussion panel, observe that the discussion title appears as:
"Submission 24 is ready to be copyedited for {$contextAcronym}."
3.The Copyeditor receives a notification stating:
"You have been asked to review copyedits for '{$file}'."

Expected Result

The discussion title should properly populate {$contextAcronym} and copyediting notification should properly populate {$file} tags.

Actual Result

The discussion title is not properly populating {$contextAcronym}, and the copyediting notification is not populating the {$file} tags.

Environment Details

No response

Application Version

OMP 3.5

Logs

No response

Additional Information

No response

@mbakioglu mbakioglu changed the title [Main Feature] | Copyediting Stage- Discussion tags do not populate Copyediting Stage- Discussion tags do not populate Mar 18, 2025
@Tribunal33 Tribunal33 self-assigned this Mar 19, 2025
@Tribunal33 Tribunal33 changed the title Copyediting Stage- Discussion tags do not populate [Submission] | Discussion - Copyediting Stage, Discussion tags do not populate Mar 19, 2025
@Tribunal33 Tribunal33 added the Bug:0:Trivial A bug mainly used for UI bugs that do not affect common functionality but break user experience. label Mar 19, 2025
@Tribunal33 Tribunal33 added this to the 3.5.0 LTS milestone Mar 19, 2025
@Tribunal33
Copy link
Contributor

Might be related to #10962 . Similar missing variable reference.

@taslangraham
Copy link
Contributor

Will take a look

@taslangraham taslangraham self-assigned this Mar 20, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 25, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 25, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 26, 2025
@taslangraham
Copy link
Contributor

taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 26, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 27, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 27, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 27, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 27, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 27, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/pkp-lib that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/ojs that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
taslangraham added a commit to taslangraham/omp that referenced this issue Mar 28, 2025
@taslangraham
Copy link
Contributor

Ready for review @asmecher @Vitaliy-1

Changes

  • Rename contextAcronym email variable to app specific variable name (e.g journalAcronym for OJS) during installation.
  • Add migration script to update existing contextAcronym email variable to app specific variable in database.
  • [OMP] remove app level locale for notification.type.copyeditorRequest which had invalid {$file} variable.
    • Locales already exists within pkp-lib
  • First 2 bullet points also fixes #10962

PRs

ojs main
ojs 3.5
ojs 3.4

omp main
omp 3.5
omp 3.4

pkp-lib main
pkp-lib 3.5
pkp-lib 3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:0:Trivial A bug mainly used for UI bugs that do not affect common functionality but break user experience.
Projects
None yet
Development

No branches or pull requests

3 participants