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.

Sign up for GitHub

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

Jump to bottom

Use Studip-Icon for ServerCard checkbox and show incoming messages always as new ones #855

Merged
merged 1 commit into from
Dec 11, 2023
Merged

Use Studip-Icon for ServerCard checkbox and show incoming messages always as new ones #855

merged 1 commit into from
Dec 11, 2023

Conversation

ssrahn
Copy link
Collaborator

@ssrahn ssrahn commented Dec 8, 2023

#832

  • Nutze Studip-Icon für Checkbox in ServerCard
  • Meldung, ob Server aktiviert oder deaktiviert wurde
  • Änderung in Messages
    • Vorher: Wenn bereits existierende Message hinzugefügt wird, wird die Message ignoriert
      • Bedeutet die Message bleibt einfach an der selben stelle stehen. So könnte es passieren, dass der User eine Meldung nicht wahrnimmt, wenn diese erneut auftritt und noch in der MessageList vorhanden ist
    • Jetzt: Lösche die existierende Message und füge sie erneut hinzu
      • So tauchen hinzugefügte Messages immer unten in der Liste auf und sind auffälliger für den User

@ssrahn ssrahn added v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:visual-clarity labels Dec 8, 2023
@ssrahn ssrahn requested a review from tgloeggl December 8, 2023 10:19
@ssrahn ssrahn self-assigned this Dec 8, 2023
@tgloeggl tgloeggl merged commit 1edeed1 into master Dec 11, 2023
5 checks passed
@tgloeggl tgloeggl deleted the fix832 branch December 14, 2023 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgloeggl tgloeggl Awaiting requested review from tgloeggl

Assignees

@ssrahn ssrahn

Labels
type:visual-clarity v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ssrahn @tgloeggl