Feature/ip 99 integracja UI powiadomien#56
Conversation
…nhance NotificationBell. Create logic to connect with backend using SSE
…ount to enhance UX. Make notifications fetched only after clicking the bell.
# Conflicts: # frontend/src/api/endpoints.ts # frontend/src/api/index.ts # frontend/src/routes/paths.ts
…ellItem Component. And add sending events in correct places on backend.
|
Trochę dużo linijek kodu wyszło ale to też wygenerowany kod i imo na backendzie nie ma co dużo sprawdzać, głownie po prostu wywołania eventów, można na szybko przejrzeć. Na froncie notyfikacje można przejrzeć hooki. Jest dzwoneczek razem z popoverem i strona notyfikacji. Myślę, że samego kodu html nie ma zbytnio co przeglądać raczej szkoda czasu. Ewentualnie użycie hooków czy jest poprawne itp. |
# Conflicts: # frontend/src/pages/EmployeeAssignmentsPage.tsx
PatrykBlacha
left a comment
PatrykBlacha
left a comment
There was a problem hiding this comment.
Wygląda wszystko super, widać że porządnie. Wydaje mi się tylko że możnaby dodać jakiś QualificationNotificationListener na backendzie czy coś takiego żeby nie pisać powiadomień na sztywno w serwisach.
| }; | ||
|
|
||
| const handleMarkAsReadOnly = (e: React.MouseEvent, id: string) => { | ||
| e.stopPropagation(); |
There was a problem hiding this comment.
czy my tego potrzebujemy? to jest chyba związane z komentarzem Patryka: https://github.com/FAZ110/IO-Project/pull/56/changes/BASE..5bff36e65951467f4df840bda0a8340dd39acaca#r3251181495
dwa handlery i musimy zatrzymać propagację przez to, gdyby poprawić tamten komponent to również można by uprościć
There was a problem hiding this comment.
No trzeba bo jest button do oznaczania jako przeczytanie w środku diva z onClick. Więc stopPropagation musi być, żeby nie odpaliła się funkcja nadrzędna.
…ooks. Extract the business logic of formatting string from services to Event records with new method buildMessage. And fixes from review.
3 participants
Stworzenie UI dla powiadomień na frontendzie. Dzwonek powiadomień i strona powiadomień. Integracja z backendem. Wywoływanie odpowidnich eventów na backendzie.