Zadanie #1806
Komponenty rejestrujące się w brokerze muszą się odrejestrowywać
| Status: | Rozwiązany | Start date: | 2017-08-18 | |
|---|---|---|---|---|
| Priority: | Normalny | Due date: | ||
| Assignee: | Michał Komar | % Done: | 100% | |
| Category: | ui | |||
| Target version: | 0.3 | |||
| Developer: | Michał Komar | Tester: | Wojtek Hury |
Description
Obecnie komponenty nie odrejestrowują swoich callbacków z brokera (dokładniej z DocumentEventService, który z kolei rejestruje callback w BrokerService). W czasie działania aplikacji może powstać więcej niż jeden EditDocumentComponent i pojawiają się problemy związane z wywoływaniem callbacków starych komponentów.
Sposób odtworzenia:- wejście na ekran edycji
- naciśnięcie "Zapisz" działa poprawnie
- wejście na ekran administracji użytkownikami
- powrót do ekranu domowego
- ponowne wejście na ekran edycji
- naciśnięcie "Zapisz" pozornie działa, ale już wywoływane są dwa callbacki
Jeśli w międzyczasie nastąpi rozłączenie sesji po stronie serwera (nawet bez potrzeby przejścia przez ekran logowania), to 'stary' callback będzie rozpoznawał zmiany dokonane na ekranie edycji jako zmiany innego użytkownika i pokazywał snackbar z ostrzeżeniem. Żeby doprowadzić do rozłączenia sesji wystarczy przytrzymać dłużej przeglądarkę na typescriptowym breakpoincie.
Associated revisions
refs #1806: Obsługa unsubscribe. Refaktoring broker-service
History
#1 Updated by Wojtek Hury over 7 years ago
- Description updated (diff)
#2 Updated by Wojtek Hury over 7 years ago
- Description updated (diff)
#3 Updated by Michał Komar over 7 years ago
- Assignee set to Michał Komar
- Target version set to 0.3
- Developer Michał Komar added
- Tester Wojtek Hury added
#4 Updated by Michał Komar over 7 years ago
- Status changed from Nowy to Testowanie
- Assignee changed from Michał Komar to Wojtek Hury
- % Done changed from 0 to 100
Dodałem możliwość unsubscribe.
Poprawiłem reconnect.
Uprościłem także kod w broker service.
#5 Updated by Wojtek Hury over 7 years ago
- Status changed from Testowanie to Rozwiązany
- Assignee changed from Wojtek Hury to Michał Komar