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

System warning #4413

Closed
wants to merge 6 commits into from
Closed

System warning #4413

wants to merge 6 commits into from

Conversation

solth
Copy link
Member

@solth solth commented May 18, 2021

This Pull Request adds the option for admins to set system messages and send nofitications to all users about, for example to inform them about upcoming maintenance work or server shutdowns.

Two methods are added to facilitate these system notications:

  1. setting a "Static system message" that is displayed to all users on all pages when it is activated, comparable to the old functionality of Kitodo 2 (see screenshots 1 & 2)
  2. sending a "Broadcast" to all users currently logged into the system via websockets. This results in a modal popup dialog shown to the users which they must acknowledge by clicking on "Ok". The admin who broadcasted the message can see which users acknowledged the message on the system message page. (see screenshot 3 & 4).

Screenshot 1: write a static system message
Bildschirmfoto 2021-05-18 um 11 57 16

Screenshot 2: static system message is displayed to all users
Bildschirmfoto 2021-05-18 um 11 58 18

Screenshot 3: write a broadcasting message to all users and monitor which users acknowledge the message:
Bildschirmfoto 2021-05-18 um 11 59 20

Screenshot 4: broadcasting message displayed to users in modal dialog to acknowledge:
Bildschirmfoto 2021-05-18 um 11 59 38

Fixes #3586

@solth solth force-pushed the system-warning branch from ca7c1b6 to 38ecde7 Compare May 19, 2021 10:14
@solth solth requested a review from Kathrin-Huber May 19, 2021 17:30
@Kathrin-Huber
Copy link
Contributor

login

The loginpage is broken (no orange footer, login without margin), because it is taken out of "base.xhtml"

@Kathrin-Huber
Copy link
Contributor

The new authority is not translated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing notification function to all users
2 participants