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

Some suggestions #25

Open
pirrimanson2000 opened this issue May 1, 2020 · 4 comments
Open

Some suggestions #25

pirrimanson2000 opened this issue May 1, 2020 · 4 comments

Comments

@pirrimanson2000
Copy link

Hi,
¡Thank you for develop this plugin!. I've been using the Publish plugin until now, but I think that the Approve plugin has some advantages over Publish.

Some things I miss:

  • I would like to disable the "Approved $DATE by $USER (version X)" message for the read only users. If not possible, at least that we can choose the location of the message (for example, at the bottom of the page)
  • I would like to change the "Approved $DATE by $USER" message (for example, to delete the $USER part or to translate the whole message).
  • I would like to have the posibilty of approving all pending changes in one click.

Bye!

@mdiedrich-rz
Copy link
Contributor

Hi,

a general quick win: You can style the display position and color of the banner by adding your styling to your local userstyle.css.

Hiding the approve message for r/o users and shortening it is basically (but fix, non-editable) implemented in https://github.com/mdiedrich-rz/dokuwiki-plugin-approve/tree/display_banner (not reviewed yet, my personal development). Comments are appreciated.

Approving all pending changes at once could be a good idea - in my opinion approving a page means viewing it, reading it, understanding it, thinking about it and then approving it (or not ;)). So batch processing isn't possible in that process and so normally not needed (perhaps for the first time after enabling the approve plugin). My approvement - my responsibility - my job to really review the changes. A question of processes to be defined :).

I'd just like to add another suggestion:

  • Time based notification that a scheduled review is neccessary (perhaps after a configurable amount of time). I'll try to implement that in my branch too.

Bye and thanks for your suggestions :).

@solewniczak
Copy link
Member

solewniczak commented May 14, 2020

The time based notifications was implemented by my as watchcycle plugin some time ago. But it would be better if this feature is integrated into approve directly.

I would like to have the posibilty of approving all pending changes in one click.

Do you want to approve all pages without viewing the changes between revisions?

@pirrimanson2000
Copy link
Author

Thanks for your answers,

About "adding your styling to your local userstyle.css.": I tried to mantain dokuwiki "as standard as possible" to avoid problems with future updates. But is an alternative, thanks.

About "approve all pages": some scenarios where I think that this option would be useful:

  • When the person who made the changes is the same that approve them, and is not possible to disable the plugin in that namespace (for example, when the plugin has been actived to comply with some regulation, in my case ISO 27001).
  • When you active the plugin for the first time
  • When you do a "batch editing" with the BatchEdit Plugin to change some strings in a lot of pages at once (not sure if this case would be detect as a new pending approval).

Publish plugin has an option for this, we use it from time to time.

@solewniczak
Copy link
Member

Sounds reasonable. I believe we should include it in future releases.

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

No branches or pull requests

3 participants