Feature/cu 861m77zun - Dettaglio Aula

[bob27aggiustatutto]

No description provided.

[PoliNetwork-ClickUp]

Task linked: CU-861m77zun Pagina Dettaglio Aula

[PoliNetwork-ClickUp]

Task linked: CU-861m3wffu Pagina Aule Libere

[DiegoZaff]

Per chi si chiedesse perché sono andato a modificare il componente CustomModal usato nel MainMenu spostando titolo e sottotitolo dal componente alla prop children è andata circa cosi:

1. ho cercato di riutilizzare il CustomModal per la sezione affollamento con lo slider dinamico, quindi mi serviva il titolo e sottotitolo in posizioni diverse da come era stato progettato il componente, quindi li ho rimossi e passati come children (in modo da avere meno limitazioni).
2. Ho realizzato che i react native gesture handlers non funzionano dentro i modali, o meglio, non funzionavano dentro il modale wrappato con Pressable, inoltre il contenuto del modale deve essere anche wrappato con una funzione particolare (vedi codice).
3. Ho creato un nuovo modale specifico per le gestures.
4. I cambiamenti nel CustomModal mi sembravano sensati e li ho lasciati come sono

[DiegoZaff]

La pagina è quasi conclusa. Manca di capire come far funzionare il componente "Mancano ... ore ... minuti" e se le aule con i nomi particolari vengono mostrate correttamente.

@toto04 vuoi fare la review prima del merge nel branch di Matteo o facciamo tutto alla fine?

[lups2000]

prima di fare la review forse sarebbe meglio fare un merge con main come è stato fatto con il mio branch per risolvere subito quella cosa degli spazi

[DiegoZaff]

Per la pagina di dettaglio aula devo ancora cambiare il componente da... a ... Per riflettere l ultimo cambiamento dell' endpoint per cui restituisce anche quando l aula è libera dopo l' orario di fine specificato. Faccio stasera

I cambiamenti di design e di backend per quanto riguarda l endpoint che restituisce le aule libere da ora fino a fine giornata e l aggiunta del picker x la data di fine non dovrebbero riguardare questa pr

[DiegoZaff]

Penso che l'endpoint abbia dei problemi perché prima mi restituiva errore 500 la chiamata per la ricerca della aule.
Comunque, la pagina di dettaglio articolo al momento dovrebbe essere funzionante e in pari con lo stato attuale degli endpoint, devo testare gli ultimi fix se non hanno rotto qualcosa.

Poi si può procedere alla review e merge.

Il collegamento con le informazioni di {occupancies, latitude e longitude} deve essere ancora fatto. Lo devo fare io?

[toto04]

ma sì dai, se hai voglia sì

[DiegoZaff]

Ho fixato alcune cose nella pagina di dettaglio aula e fatto il collegamento con le informazioni reali tramite la scelta con Campus -> Edificio -> Aula.

Momentaneamente ho messo che cliccando sulle notifiche si apre una pagina con dati hardcodati per mostrarne lo stato attuale, dato che l'endpoint è stato fixato e non restituisce più aule libere dopo le 20:00. Dovremmo valutare dei messaggi di errore o qualcosa da mostrare se non vengono restituite aule. Anche mettere degli activity indicator mentre le aule vengono ricercate (dato che ci può mettere diversi secondi).

Per informazione i dati di quell'aula sono:

            room: {
              name: "2.0.1",
              capacity: 380,
              building: "Edificio 2",
              address: "Piazza Leonardo da Vinci, 32 - 20133 - Milano (MI)",
              power: false,
            },
            startDate: "2023-02-22T16:15:00Z",
            roomId: 32,
            roomLatitude: 45.4788249919485,
            roomLongitude: 9.227210008150676,
            occupancies: {
              // eslint-disable-next-line @typescript-eslint/naming-convention
              "19:00": "FREE",
              // eslint-disable-next-line @typescript-eslint/naming-convention
              "19:35": "OCCUPIED",
            },
          }

[toto04]

io direi che questo potrebbe avere un alias dato che lo continui a usare ovunque, puoi anche eventualmente essere più specifico con il key type usando template literals dato che hanno una forma precisa

export type Occupancies = Record<`${number}:${number}`, "FREE" | "OCCUPIED">

[DiegoZaff]

Ok va bene. Devo aggiungere un po' di codice nella funzione getEndDate() per chiamare occupancies[key] ma niente di assurdo. Poi quando faccio il commit vedi.

[toto04]

Probabilmente questo dovrebbe diventare un colore nel ColorTheme della palette, qualsiasi colore che usi più di una volta dovrebbe stare lì, puoi anche chiamarlo qualcosa come sliderBorderColor, o come preferisci, dipende da dove lo riutilizzi ma ho visto che in giro ritorna ogni tanto

[DiegoZaff]

Ok, penso di aver fatto un po' di confusione con i colori dei vari componenti e di non aver usato l'hook ovunque, faccio un controllo anche in tutti gli altri file

[toto04]

idem, nella palette

[DiegoZaff]

ok

[toto04]

La definizione di questo potresti portarla qui dentro e poi esportarla, se no fai import circolari che non è che siano un problema finché sono solo di tipi perché poi vengono tolti da tsc, ma fa brutto lo stesso ahah

ma on another note, serve? perché da quanto ho capito l'endpoint restituisce un float

[DiegoZaff]

Non so per quale motivo pensavo dovesse restituire un intero compreso tra 1 e 5. Quindi se può restituire anche numeri con la virgola devo fare parecchi cambiamenti.

[toto04]

tutti questi dovrebbero stare nella palette, sono tutti uguali

[toto04]

sì direi che ha senso, non è che sia molto utile fare pagewrapper per ogni sezione ahah, andrebbero unificate, cercando di mantenere il resto del codice invariato e aggiungendo solo i mezzi per ottenere le differenze per ogni pagina, vedi tu se riesci a farlo in modo poco invasivo, se devi modificare troppa roba allora diventerà una issue futura

[DiegoZaff]

Ok

[toto04]

you know it, palette, quantomeno usare i colori della palette tipo

isLight ? palette.primary : palette.lighter

[DiegoZaff]

Ok

[toto04]

Ok allora mi sembra tutto a posto, ovviamente adesso @lups2000 dovrai occuparti di rifinire il tutto assicurandoti che funzioni tutto senza problemi, ci potrebbe volere del tempo e te e @DiegoZaff eventualmente discutete direttamente tra di voi in caso per quanto riguarda l'interfaccia tra queste pagine
glhf :)

[github-actions]

Updated task status CU-861m77zun to: completed

[DiegoZaff]

Va bene, mi metto d'accordo con Matteo per fare eventualmente modifiche nel suo branch.