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

[Notifications] Improve notification list and notifications. #2410

Open
MagpieFourtyTwo opened this issue Apr 26, 2023 · 5 comments
Open

[Notifications] Improve notification list and notifications. #2410

MagpieFourtyTwo opened this issue Apr 26, 2023 · 5 comments
Assignees

Comments

@MagpieFourtyTwo
Copy link

Is your feature request related to a problem? Please describe

Creating notifications is always a real PITA:

  • No way to change just one thing (e. g. the cache type) and then say Save as new,
    which appears to be the standard, because you usually don't want to be notified about new Tradis only,
    but also for Multis, Mysteries, Letterboxes, Virtuals, ECs, WIGs ...,
    which means you're 7 times forced to start over from scratch (Event types not even counted).
  • Even more annoying, as there is no possibility to copy/paste coordinates, as there are 6(!) fields to be satisfied
    instead of just one, to be able to just copy/paste the entire coordinate string
  • And of course because there's no poss. to create notifications for more than just one cache type in one go

Moreover it's kind of hard to check the existing notifications:

  • in the list of notifications there are no coordinates
  • same applies for configured radius
  • no possibility to view the existing notifications on a map

And things get even worse if you're going to create (and maintain) notifications for a tour ... to e. g. cover a route of 100 km for the typical "on the road cache types", you have to set up 7 notifications, with 15 km radius each, every 20 km. Which in fact makes 7*5=35 walks through this ugly form. Not daily business, of course, but nevertheless horrible, if you want to do so (and if your way to work changes weekly).

Describe the solution you'd like

My wish list for notification creation:

  • coordinates: One input field to be able to paste a coordinate string (would already help A LOT)
  • button Save as New (would be really great)
  • tick list for cache types, to be able to create notifications for more than one type in one go,
    using the same center and radius (would be absolutely perfect ;) )

The wish list for the notification list:

  • show center coordinates
  • show radius
  • possibility to visualize notification circles on a map

Additional context

Again, BIG thanks for this GREAT piece of software which not only makes gc.com a MUCH better place, but in fact makes it usable at all. Our hobby would be way less fun without it.

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

2Abendsegler commented Nov 10, 2023

@MagpieFourtyTwo
Moin,

ich hatte die letzten Monate immer mal wieder an Verbesserungen für die Notification List und die Notifications gearbeitet. Leider konnte ich keine API zum Lesen oder Speichern finden. Und leider ist die Notification Seite so beschaffen, dass wir beispielsweise zum Wechseln der Cache Typen und auch zum Speichern irgendwelcher Änderungen immer online über die Seite gehen müssen. Die Felder und Buttons müssen also auf der Seite ausgewählt werden, so als ob ein User das selbst macht.

Bis zu einem bestimmten Grad können wir eine Seitenfolge über das Script aufrufen und die Seiten durch das Script füllen lassen. Ein Ablauf derart, dass man beispielsweise 8 Cache Typen auswählt und dann per einmaligem Drücken eines neuen "Save as new" Buttons mit den vorhandenen Werten 8 neue Notifications über 24 Seitenfolgen anlegt, kann ich mir nicht vorstellen. Ein solcher Ablauf ist nicht mehr DAU kompatibel.

Was wir aber machen können sind einzelne Abläufe per "Save as new" Button oder ähnliches.

Der "Save as new" Prozess ist noch nicht ganz fertig und ich weiß auch nicht ob ich bei den ganzen Arbeiten im Rahmen des Redesigns für das neue Log Formular noch Zeit finde, den Prozess für die nächste Version fertig zu machen. Es gibt aber einige andere Hilfen, mit denen ich schon etwas weiter bin, obwohl auch die noch nicht komplett durch getestet sind.

Diese Hilfen habe ich mal zusammengestellt und kann sie auch hier zum Testen zur Verfügung stellen. Wenn jemand Zeit und Lust hat richtig zu testen, dann könnten wir zumindest diese Teile vielleicht noch in die nächste Version packen.

Die Notification List wurde etwas aufgehübscht und hat neue bzw. verbesserte Features erhalten:

  • Aktivieren/Deaktivieren Notification ohne Reload der Seite.
  • Kopieren der Daten einer Notification.
  • Löschen einer Notification per Klick automatisch über Popups.
  • Alle Notification Daten laden, anzeigen, ausblenden.
  • Seiten Info ausblenden, einblenden.
  • Sortierung der Spalten durch Klick auf Spaltenkopf. [Edit]

Notifiction List

Auf der Seite der Notification gibt es auch Neues:

  • Kopieren der Daten einer Notification, um dann auf dieser Seite nach manueller Anpassung ein neue Notification anlegen zu können. Ähnlich "Save as new" nur mit Ausgangspunkt in der Notification Liste.
  • Koordinateneingabe über ein einziges Feld.
  • Speichern mit F2 Taste. [Edit]

Notifiction

Hast du vielleicht in den nächsten Tagen etwas Zeit zum Testen?

@MagpieFourtyTwo
Copy link
Author

Moin!

Zeit zum Testen ist aktuell leider ziemlich knapp, aber ich kann's natürlich trotzdem versuchen.
Andererseits möchte ich Deine Kapazitäten aber nicht allzu sehr an dieses quasi "Bequemlichkeitsproblem" binden,
ich denke, das neue Logformular (das ich noch viel grauslicher finde als die Notifications) ist da deutlich wichtiger
und vor allem auch für deutlich mehr Leute "wichtig". Von daher: Gerne, ja, aber lieder erst mal wieder vernünftig loggen. ;)
Bzw. aktuell natürlich erst mal "nur" editieren ... aber das Draft-Logformular kann ja gleich das nächste sein ... :/

@MagpieFourtyTwo
Copy link
Author

Gerade die neue Version avisiert bekommen, Changelog gelesen, angesehen, prima, prima, ich bin begeistert: :)
Copy ist ne coole Sache, 1-Feld-Coord-Eingabe ebenfalls, die Tabellenansicht allerdings ... da dachte ich zuerst, die tut nicht.

Bis ich mich an das "Load additional data" erinnert habe. - das könnte vielen entgehen, fürchte ich ...
Daher die Frage: Was ist der Grund für das initiale Verbergen? Geschwindigkeit wohl eher nicht,
das Einblenden ging bei mir trotz 38 Notifications (was eher viel sein dürfte) ohne nennenswerte Verzögerung.
Unmittelbar nach dem Klick waren die meisten sofort da und der letzte nach max. einer halben Sekunde.

Ich persönlich würde daher initial alles zeigen (damit überhaupt bemerkt wird, dass da was ist)
und entweder den Initial-Zustand per Option ändern lassen oder (verm. besser) die letzte Einstellung merken.

Grundsätzlich aber wieder mal genial, was ihr mit der neuen Version auf die Beine gestellt habt. Rrreschbeggd!
Das macht das Cacher- und vor allem auch Logger-Leben wieder ein gutes Stück einfacher. DICKES Danke! ;)

@2Abendsegler
Copy link
Owner

Was ist der Grund für das initiale Verbergen?

Wir versuchen die Server von GS möglichst nicht mit vielen Anfragen zu nerven. Zudem ist es eigentlich gar nicht erlaubt. Wenn wir Daten anfragen, dann machen wir das möglichst nur in überschaubarem Rahmen. Wir fühlen uns dann wohl, bleiben etwas im Hintergrund und wir werden auch nicht aufgefordert irgendwelche Anfragen zu unterlassen.

Bei einem Log fragen wir einmal das Listing an. Bei den Pocket Queries fragen wir sogar 20 mal die Favoriten Prozente an, allerdings ist die Datengröße nur minimal. Bei den BML wären die Favoriten Prozente auch nett, allerdings wären das unter Umständen 500 Anfragen. Das haben wir nicht realisiert. Nur um einige andere Beispiele zu nennen.

Für eine Notification müssen wir im Hintergrund die Webseite der Notification laden, weil es keine API gibt. Bei dir also 38 Anfragen. Würden wir davon ausgehen, dass jeder 38 Notifications in seiner Liste hat, hätten wir das Laden vermutlich nicht entwickelt. Wir gehen aber auch davon aus, dass es im Durchschnitt eher etwas weniger sind. Unabhängig davon werden viele User die zusätzlichen Daten gar nicht benötigen, oder zumindest nicht immer. Also sollen sie klicken, wenn sie die Daten benötigen und es lassen, wenn sie die Daten nicht brauchen. Das ist nur ein Klick und die Daten sind ja auch zügig vorhanden.

Danke für die Blumen! 😊

@MagpieFourtyTwo
Copy link
Author

Verstehe - nachvollziehbar.

Danke für die Blumen! 😊

Ehre, wem Ehre gebührt. ;)

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

No branches or pull requests

2 participants