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

[Notification] Improve notification list and notifications. #2454

Merged
merged 11 commits into from
Nov 24, 2023

Conversation

2Abendsegler
Copy link
Owner

@2Abendsegler
Copy link
Owner Author

@capoaira
Ich schon wieder. 🙄

Falls du magst, kannst du hier mal drüber schauen. Ich werde es als Beta in die nächste Version packen. Bitte aber nicht approven.

@2Abendsegler 2Abendsegler self-assigned this Nov 17, 2023
@2Abendsegler 2Abendsegler added this to the v0.14.7 milestone Nov 17, 2023
@2Abendsegler 2Abendsegler changed the title Improve notification list and notifications. [Notifications] Improve notification list and notifications. Nov 17, 2023
@capoaira
Copy link
Collaborator

Wow, ich finde, das sieht schon richtig gut aus 👍👍👍

Ich habe noch zwei Sachen gefunden, die ich schön finden würde und einen kleinen Bug.

  1. Es wäre schön, wenn beim Ändern des Cachetypes, die Logtypen erhalten bleiben (sofern möglich). So kann man das Copyfeature effizienter nutzten

  2. Es wäre schön, wenn man Notification mit demselben Namen gleichzeitig (de)aktivieren könnte. (z.B. mit Rechtsklick, so wie man auch bei den PQs mehrere gleichzeitig (de)aktivieren kann)

  3. Wenn man eine Notification löscht, wird sie ausgegraut. Wenn sich endgültig gelöscht wird, ist jedoch immer noch zu sehen. Hier wäre es sinnvoll, entweder die Notification dann verschwinden zu lassen oder einen "rückgängig" Button (bzw. den "bearbeiten" Button) anzuzeigen, mit der man die Notification wieder herstellen könnte.

Soweit mein erster Eindruck, ich habe noch nicht strategisch alle Funktionen abgearbeitet, sondern nur meinen eigenen Workflow, beim Erstellen neuer getestet.

Ich finde die Erweiterungen klasse und werde sie bei meinem nächsten Urlaub auf jeden Fall nutzen 😊👍

@2Abendsegler
Copy link
Owner Author

Alles super Verbesserungsvorschläge!

  1. versuche ich noch einzubauen. 3. vielleicht auch. 2. Kommt wohl erst später.

@2Abendsegler 2Abendsegler marked this pull request as draft November 18, 2023 15:28
@2Abendsegler 2Abendsegler modified the milestones: v0.15 [Log Form], v0.15 Nov 23, 2023
@2Abendsegler
Copy link
Owner Author

@capoaira
Ich schon wieder. 🙄

  1. Es wäre schön, wenn beim Ändern des Cachetypes, die Logtypen erhalten bleiben (sofern möglich). So kann man das Copyfeature effizienter nutzten

  2. Es wäre schön, wenn man Notification mit demselben Namen gleichzeitig (de)aktivieren könnte. (z.B. mit Rechtsklick, so wie man auch bei den PQs mehrere gleichzeitig (de)aktivieren kann)

  3. Wenn man eine Notification löscht, wird sie ausgegraut. Wenn sich endgültig gelöscht wird, ist jedoch immer noch zu sehen. Hier wäre es sinnvoll, entweder die Notification dann verschwinden zu lassen oder einen "rückgängig" Button (bzw. den "bearbeiten" Button) anzuzeigen, mit der man die Notification wieder herstellen könnte.

Punkt 1. habe ich eingebaut. Deine beiden anderen Punkte schaffen es nicht mehr in die nächste Version. Den Delete Process habe ich für die nächste Version komplett raus genommen.

Ich habe gut getestet. Vielleicht hast du aber trotzdem noch mal Zeit drüber zu schauen. Wie erwähnt, will ich spätestens am Samstag Abend eine neue Version bringen.

  • Hier steht im Groben welche Features bisher eingebaut wurden.
  • Es sind Anpassungen im init.js hier im PR zu beachten.

@2Abendsegler 2Abendsegler marked this pull request as ready for review November 23, 2023 21:58
Copy link
Collaborator

@capoaira capoaira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht wirklich alles Klasse aus. 👍

Mir ist nur noch eines aufgefallen: Bei mir wird die E-Mail nicht angezeigt:
grafik

Ich gehe davon aus, dass es daran liegt, dass ich nur eine angegeben habe und daher kein select zur Verfügung steht.

@2Abendsegler
Copy link
Owner Author

@capoaira
Super!
Passt auch bei mir.
Ich merge dann.

@2Abendsegler 2Abendsegler merged commit 210f068 into collector Nov 24, 2023
@2Abendsegler 2Abendsegler changed the title [Notifications] Improve notification list and notifications. [Notification] Improve notification list and notifications. Nov 25, 2023
@2Abendsegler 2Abendsegler deleted the improve-notifications branch December 8, 2023 21:34
2Abendsegler added a commit that referenced this pull request Dec 14, 2023
… with same name.

Das ist Vorschlag 2. aus PR #2454.
> Es wäre schön, wenn man Notification mit demselben Namen gleichzeitig (de)aktivieren könnte. (z.B. mit Rechtsklick, so wie man auch bei den PQs mehrere gleichzeitig (de)aktivieren kann)
2Abendsegler added a commit that referenced this pull request Dec 14, 2023
… with same name.

Das ist Vorschlag 2. aus PR #2454.
> Es wäre schön, wenn man Notification mit demselben Namen gleichzeitig (de)aktivieren könnte. (z.B. mit Rechtsklick, so wie man auch bei den PQs mehrere gleichzeitig (de)aktivieren kann)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants