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

Merges the Typesetting Plugin in to the core of Janeway #4612

Open
wants to merge 346 commits into
base: master
Choose a base branch
from

Conversation

ajrbyers
Copy link
Member

@ajrbyers ajrbyers commented Feb 5, 2025

The aim of this PR is not to overhaul anything but simply to make the Typesetting plugin a core Janeway application.

A brief summary:

  • Moves the Typesetting plugin into core and maintains its git history
  • Maintains the plugins/typesetting/ URL path but adds typesetting/ too
  • Renames the workflow element from Typesetting Plugin to typesetting for consistency
  • Updates the default workflow to include Typesetting, removes production and proofing
  • Plugin loading will ignore the Typesetting plugin
  • Merges events and settings into core modules

ajrbyers and others added 30 commits March 17, 2020 10:10
Corrections can be added on any typesetting round where a galley is
present.
- Typesetters will see a warning until they have done all corrections
- Managers will receive a prompt to act upon corrections not done
- Managers now have access to proofreader notes when creating rounds
* Initial email commit.

* More work on emails!

* Closes #19

* Moves notification stuff to its own module.

* Made some changes. 

Logging is done in model method where possible, event raise from notify.py

* See note:

Splits notify and event firing. Views/logic fire events via notify and events registered funcs are defined in emails.py

* Addresses comment 

openlibhums/typesetting#29 (review)

* Addresses comment

openlibhums/typesetting#29 (comment)

* Set user=None where not explicitly required.
* Completing a task marks it as accepted.

* Added check to see if there is an existing accepted date.

* Moved journal filter to initial query.
@ajrbyers
Copy link
Member Author

ajrbyers commented Feb 5, 2025

  • Migrate the hook to core.

@mauromsl mauromsl added this to the v1.8.0 Tracker milestone Feb 6, 2025
@S-Haime S-Haime added the documentation Ideas for new or improved user documentation label Feb 6, 2025
@S-Haime
Copy link
Member

S-Haime commented Feb 6, 2025

Feel free to assign to me post merge. I can then check docs to make sure this reflects it. I may sit down with someone to check I ain't miss anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Ideas for new or improved user documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants