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

Journal Editor Restriction plugin #277

Merged
merged 3 commits into from
Jul 15, 2024

Conversation

rahmanramsi
Copy link

@rahmanramsi rahmanramsi commented May 2, 2024

This plugin will disable access for Journal Editors to the following sections:

  • Journal Settings
  • Website Settings
  • Workflow Settings
  • Distribution Settings
  • Users & Roles
  • Tools

This feature is implemented to be similar to the editor workflow system in OJS 2

Access the documentation here : https://openjournaltheme.com/docs/ojs-product-documentation-free/journal-editor-restriction-plugin/

Access the repository here : https://github.com/openjournalteam/journalEditorRestriction

@asmecher
Copy link
Member

asmecher commented May 2, 2024

Thanks, @rahmanramsi! I've made some suggestions in the plugin's repo: openjournalteam/journalEditorRestriction@a36d177

@rahmanramsi
Copy link
Author

Thanks for the suggestion @asmecher.
Suggestions will be implemented within the next week based on my workload.

@rahmanramsi
Copy link
Author

Hi @asmecher, i have implemented your suggestions into code, can you review it ?
Here is the PR : openjournalteam/journalEditorRestriction#5

plugins.xml Show resolved Hide resolved
@asmecher
Copy link
Member

asmecher commented Jul 9, 2024

@rahmanramsi, thanks, that looks good to me. I've added a couple trivial comments but nothing needing urgent attention. If you'd like to wrap up a new .tar.gz package and update this PR, I can get it merged!

@rahmanramsi
Copy link
Author

@asmecher updated the tar.gz

@asmecher asmecher merged commit 4aba852 into pkp:main Jul 15, 2024
1 check passed
@asmecher
Copy link
Member

Merged -- thanks, @rahmanramsi, and sorry for the delay!

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