-
Notifications
You must be signed in to change notification settings - Fork 6
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
GO-23 Automaticky zobraziť prevzatú správu #475
Comments
@stage-rl ak by si mal priestor na nejaku ulohu, skusil by si sa na toto pozriet prosim? |
No, pisete sice explicitne Action Cable, pre istotu ale este otazka @luciajanikova @jsuchal , ked uz je Rails 8 predo dvermi, ci neist rovno do morphingu a cieloveho Rails 8 riesenia. Dikes |
morphing prave tuto zmysel uplne nedava, lebo ty nechces vidiet ako sa ti to meni pod rukami, ale byt upozorneny, ze nieco sa tam udialo. protipriklad: otvorim si vlakno, dam prevziat spravu. toto bezi async. Ak niekde na koniec vlakna pridas nejaku spravu, tak si to vlastne nevsimnes lebo to je "below fold". Ovela vacsi zmysel dava nejaky notify podobne ako trebars twitter ("new messages") alebo nieco take. |
OK, blbo som sa opytal, co som myslel, je Turbo 8. A teda nemusime vlastne na nic cakat, lebo to uz mame. T.j. kombinacia https://rubydoc.info/github/hotwired/turbo-rails/Turbo/Broadcastable a https://rubydoc.info/github/hotwired/turbo-rails/main/Turbo%2FStreamsHelper:turbo_stream_from Chapem, ze poziadavka je buchnut do cela nejakou notifikaciou, na ktoru sa da kliknut |
@stage-rl pridala som IP do zoznamu povolenych kvoli stahovaniu sprav + som do DEV schranky (ktora je v seede) naposielala nejake spravy na prevzatie. |
Akutuálne treba buď refresh vlákna alebo prejsť na zoznam správ aby sa prevzatá správa zobrazila po autorizácii doručenky (správa do vlastných rúk). Automaticky sa to nerefreshne/nezobrazí.
Chceme to ale zmeniť, implementácia by mala byť cez Action Cable. Po každom ProcessMessageJobe by išiel event na refresh (broadcast page refresh na konkrétny thread).
The text was updated successfully, but these errors were encountered: