Problemi sulle NoticeSignActions

alessandrocaseti commented 5 months ago

[x] su alcune comunicazione viene visualizzata l'infobar con severity error ma senza nessun tasto sottostante
[ ] premendo il tasto "unisciti" del flyout l'app va in crash (riaprendo l'app quella comunicazione viene segnata come aderita)
[x] a volte i tasti conferma e firma / rifiuta non vengono mostrati (e manco l'infobar)

Gabboxl commented 5 months ago

se la severity è error vuol dire che è stata rifiutata, se non è così mi dovresti fornire il caso specifico. se non viene mostrato nulla allora non è richiesta la conferma

il crash è da risolvere, però mi servono i dati della api restituiti a te, purtroppo da me non riesco a riprodurre il problema

ghost commented 4 months ago

@alessandrocaseti Buongiorno! Avrei bisogno di parlarti, in merito a questa richiesta. Posso parlarti in privato? Mi scrivi alla mia mail: padovesi.roberto@gmail.com? Grazie!

Gabboxl commented 4 months ago

@alessandrocaseti Buongiorno! Avrei bisogno di parlarti, in merito a questa richiesta. Posso parlarti in privato? Mi scrivi alla mia mail: padovesi.roberto@gmail.com? Grazie!

Ciao, ciò di cui hai intenzione di parlare è qualcosa inerente a questo progetto? Se sì ti chiedo di parlarne in pubblico in questo issue, così anche altre persone potranno contribuire al discorso

ghost commented 4 months ago

Salve! No, non è niente di sospetto. Riguarda la scuola e il registro.

Il Lun 6 Mag 2024, 17:51 Gabriele @.***> ha scritto:

@alessandrocaseti https://github.com/alessandrocaseti Buongiorno! Avrei bisogno di parlarti, in merito a questa richiesta. Posso parlarti in privato? Mi scrivi alla mia mail: @.***? Grazie!

Ciao, questo messaggio sembra un po' sospetto. È qualcosa inerente a questo progetto? Se sì ti chiedo di parlarne in pubblico in questo issue

— Reply to this email directly, view it on GitHub https://github.com/Gabboxl/ClassevivaPCTO/issues/265#issuecomment-2096369326, or unsubscribe https://github.com/notifications/unsubscribe-auth/BC4UFMFRWRRT4GDKB4VPKPLZA6RH7AVCNFSM6AAAAABG4BWKCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGM3DSMZSGY . You are receiving this because you commented.Message ID: @.***>

Gabboxl commented 4 months ago

Salve! No, non è niente di sospetto. Riguarda la scuola e il registro.

Il Lun 6 Mag 2024, 17:51 Gabriele @.***> ha scritto:

@alessandrocaseti https://github.com/alessandrocaseti Buongiorno! Avrei bisogno di parlarti, in merito a questa richiesta. Posso parlarti in privato? Mi scrivi alla mia mail: @.***? Grazie!

Ciao, questo messaggio sembra un po' sospetto. È qualcosa inerente a questo progetto? Se sì ti chiedo di parlarne in pubblico in questo issue

— Reply to this email directly, view it on GitHub https://github.com/Gabboxl/ClassevivaPCTO/issues/265#issuecomment-2096369326, or unsubscribe https://github.com/notifications/unsubscribe-auth/BC4UFMFRWRRT4GDKB4VPKPLZA6RH7AVCNFSM6AAAAABG4BWKCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGM3DSMZSGY . You are receiving this because you commented.Message ID: @.***>

Okay, se ciò di cui vuoi parlare è inerente a questo progetto ti chiederei di farlo sotto questo issue, così tutti possono contribuire al discorso

Gabboxl commented 4 months ago

@alessandrocaseti ma alla fine ti ha scritto qualcosa questo tizio? sembra avere eliminato l'account

alessandrocaseti commented 4 months ago

@alessandrocaseti ma alla fine ti ha scritto qualcosa questo tizio? sembra avere eliminato l'account

no, io non gli ho scritto perché era palese uno scam

Gabboxl commented 4 months ago

@alessandrocaseti potresti controllare se con le ultime modifiche hai ancora questi problemi?

alessandrocaseti commented 4 months ago

@alessandrocaseti potresti controllare se con le ultime modifiche hai ancora questi problemi?

ok, domani ci guardo che oggi non riesco

alessandrocaseti commented 4 months ago

@Gabboxl allora c'è molto casino sulle strings, sto correggendo e tra poco apro una pr. Apparte questo, penso che il tutto si potrebbe sviluppare meglio, avevo pensato ad un altro sistema per gestire il tutto:

usiamo una sola infobar, senza dividere sign e join

if(solo sign) text = "Per questa.... conferma di accettazione" signbutton = visible joinbutton = collapsed

if(solo join) text = "Per questa.... conferma di adesione" signbutton = collapsed joinbutton = visible

if(sing && join) text = "Per questa.... conferma di accettazione e adesione" signbutton = visible joinbutton = visible

apparte questo, attualmente i sign button non mi appaiono affatto (la stessa notice da mobile li ha)

cliccando il tasto unisciti l'app si freeza

alessandrocaseti commented 4 months ago

per il resto, io pensavo di seguire lo stesso sitema dell'app mobile:

lasciamo l'infobar con i testi come scritto sopra

il label dei tasti tipo unisciti diventa "aderito" in caso di adesione e il button passa in stato disabled

per le altre due opzioni, ci sono due casi

if(refused) button conferma = collapsed button refuse = disabled refuse.label = "rifiutata"

if(signed) button refuse = collapsed button sign = disabled sign.label = "confrmata e firmata"

alessandrocaseti commented 4 months ago

apparte questo, attualmente i sign button non mi appaiono affatto (la stessa notice da mobile li ha)

rettifico, era dovuto al fatto che hai messo la severity error anche se non rifiutata. fixato in #268

quindi, manca va solo fixata la cosa del crash

Gabboxl commented 4 months ago

@alessandrocaseti potresti controllare la situazione con le ultime modifiche per favore?

alessandrocaseti commented 4 months ago

@Gabboxl ok ora funzionano le severity / visibility, ma resta il fatto che cliccando qualsiasi bottone l'app va in crash. Tocca gestire la cosa, tipo una volta cliccato si chiude il flyout e si chiude la comunicazione + refresh della listview

Gabboxl commented 4 months ago

faremo in modo che lo stato si aggiorni senza chiudere la comunicazione

alessandrocaseti commented 4 months ago

faremo in modo che lo stato si aggiorni senza chiudere la comunicazione

sarebbe meglio, l'importante è che non va in crash

Gabboxl commented 4 months ago

faremo in modo che lo stato si aggiorni senza chiudere la comunicazione

sarebbe meglio, l'importante è che non va in crash

In realtà non so perché crashi l'app a te, dovrei fare un debug con la tua comunicazione, da me non succede nulla

alessandrocaseti commented 4 months ago

faremo in modo che lo stato si aggiorni senza chiudere la comunicazione

sarebbe meglio, l'importante è che non va in crash

In realtà non so perché crashi l'app a te, dovrei fare un debug con la tua comunicazione, da me non succede nulla

scusa ma quando aderisci, confermi o rifiuti cosa fa l'app? da me non fa nulla però non posso più toccare niente, si freeza e manco si chiude il flyout

Gabboxl commented 4 months ago

faremo in modo che lo stato si aggiorni senza chiudere la comunicazione

sarebbe meglio, l'importante è che non va in crash

In realtà non so perché crashi l'app a te, dovrei fare un debug con la tua comunicazione, da me non succede nulla

scusa ma quando aderisci, confermi o rifiuti cosa fa l'app? da me non fa nulla però non posso più toccare niente, si freeza e manco si chiude il flyout

Da me l'app rimane responsiva, sicuramente abbiamo parametri della comunicazione diversi

alessandrocaseti commented 3 months ago

@Gabboxl come faccio a farti vedere i parametri della comunicazione?

Gabboxl / ClassevivaPCTO

Problemi sulle NoticeSignActions #265