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

MESSAGE: Performance bei Berechnung der Anzahl Empfänger #164

Open
carlobeltrame opened this issue May 5, 2021 · 0 comments
Open

MESSAGE: Performance bei Berechnung der Anzahl Empfänger #164

carlobeltrame opened this issue May 5, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@carlobeltrame
Copy link
Member

carlobeltrame commented May 5, 2021

Aktuelles Verhalten

Bei Briefen und SMS, die sehr viele Empfänger haben, kann es nur schon lange dauern, die Anzahl Empfänger für den Tooltip des Vorschau- / PDF-Buttons zu berechnen. Die Anzahl die im Tooltip wird dabei bei jedem Seitenaufruf neu berechnet, und die Berechnung ist kompliziert weil sie wegen verschiedenen Arten von Subscriptions und wegen dem Adress-Gültigkeits-Check nicht in der Datenbank gemacht wird.

Gewünschtes Verhalten

Die Anzahl Empfänger ändert sich tendentiell selten, und ist allen unversendeten Briefen und SMS gemeinsam. Sie kann daher gecached werden, damit die Detailansicht schneller laden kann. Zumindest auf dem Abo sollte ein "Aktualisieren" Button vorhanden sein, mit dem der Benutzer die Berechnung erneut anstossen kann.

Schritte zum Reproduzieren

  1. Auf ein Abo mit sehr vielen (tausenden) Empfängern gehen
  2. Auf dem Nachrichten-Tab einen Brief oder ein SMS erstellen
  3. Auf die Detailansicht dieses Briefs bzw. SMS navigieren
  4. Die Seite braucht lang um zu laden

Tech-Spec

  • Bei bereits versendeten Briefen / SMS cachen wir die Anzahl Empfänger auf dem Brief / SMS (siehe auch MESSAGE: Ansicht Brief Verabeitung #152)
  • Es müssen folgende Anzahlen vorberechnet und gecached werden: Anzahl Empfänger des Abos total, Anzahl Empfänger mit gültiger Adresse, Anzahl Empfänger mit gültiger Telefonnummer, Anzahl Empfänger mit gültiger Mailadresse.
@carlobeltrame carlobeltrame added the bug Something isn't working label May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant