numerique-gouv / b3desk

BBB frontend by the French Ministry of Education
https://b3desk.readthedocs.io
Other
8 stars 1 forks source link

URL questionnaire de fin de session à déplacer en fichier config #122

Open klorydryk opened 5 months ago

klorydryk commented 5 months ago

L'url est pour le moment en BD, ce qui ne semble pas avoir d'intérêt, et pose particulièrement problème pour les anciennes salles de visio qui restent sur l'ancienne adresse. A déplacer dans le fichier de config.

LoanR commented 4 months ago

Hello @klorydryk, Cette feature a déjà été implémentée dans la #12 avec la pr #30 On retrouve la variable dans la conf https://b3desk.readthedocs.io/fr/latest/maintainers/settings.html#b3desk.settings.MainSettings.SATISFACTION_POLL_URL

Si c'est toujours en base de données, il va falloir nettoyer de votre côté, mais ça ne devrait avoir aucune incidence sur le questionnaire de satisfaction

klorydryk commented 4 months ago

Ca semble bien avoir une incidence, les anciennes salles renvoyant vers l'ancienne url, les nouvelles vers la nouvelle url. S'il y a une donnée dans la BD elle prend cette donnée, quand il n'y a rien elle prend la variable de la conf ?

Sans titre

azmeuk commented 4 months ago

D'accord, on s’était mal compris et on pensait au formulaire de satisfaction affiché sous la liste des salles. Effectivement l'url de redirection post déconnexion est actuellement enregistrée en base : https://github.com/numerique-gouv/b3desk/blob/357f3024590ad4d3725819bb0900d5b0b738efe4/web/b3desk/models/meetings.py#L90 Et effectivement, il y a un paramètre de configuration MEETING_LOGOUT_URL qui n'est utilisé que comme valeur par défaut dans le formulaire.

On peut faire en sorte que ce paramètre de configuration devienne l'url par défaut pour tous les meetings.

LoanR commented 4 months ago

Pour confirmer @klorydryk, on veut que l'url de fin de meeting soit configurable et pas sauvegardée sur les meetings ?

Ça voudrait aussi dire qu'on ne veut plus pouvoir éditer ce lien dans la personnalisation ? Que la page de fin de meeting soit toujours celle précisée dans la conf ? Screenshot from 2024-04-11 12-17-59