search

AgID / wai-portal

Application portal for Web Analytics Italia
https://webanalytics.italia.it
GNU Affero General Public License v3.0
11 stars 3 forks source link

Review modelli backend #15

Closed pdavide closed 5 years ago

pdavide commented 5 years ago

Dipende da in parte da #22.

Valair commented 5 years ago

Se sei d'accordo pensavo di usare questa libreria laravel-enum per gestire i tipi di website; per questo caso, eviterei di avere un enum scolpito nel db.

pdavide commented 5 years ago

LGTM

Valair commented 5 years ago

Solo una conferma su un dettaglio. Il seeder del portale crea un utente "super-admin", senza password di analytics dato che la password su matomo per il super-admin è creata al setup del container. Questo comporta che non potrà usufruire del SSO, è corretto?

pdavide commented 5 years ago

Si, per il momento è così. Appena discuterò un po' meglio i casi d'uso del super-admin eventualmente si potranno apportare modifiche.

pdavide commented 5 years ago

Come da discussione nella #42, il modello User dovrebbe avere avere un campo uuid (v4) per la sua identificazione al posto dell'attuale id. Questo permetterà di avere un valore più "anonimo" negli URL connessi alla gestione degli utenti (operazioni CRUD e verifica email). Per la generazione del valore del campo uuid si potrebbe usare il package laravel-uuid, come suggerito da @Valair.

pdavide commented 5 years ago

Il cambio di chiave primaria da id a uuid potrebbe impattare su molti aspetti dell'applicazione (ad esempio sulle definizioni delle relazioni tra modelli). Potrebbe essere una buona strada quella di lasciare comunque il default id e pensare al campo uuid come un'aggiunta.