Development #44756
cacher backoffice/settings/emails/options
0%
Description
Parce que trop de gens y vont et qu'en fait tout est piloté par hobo et ça sert à rien de venir ici.
Révisions associées
settings: mark options set by hobo as readonly (#44756)
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
Mais j'aurais aimé y mettre la liste des domaines connus mais à ne pas suggérer, comme noté dans #44663#note-8.
Mais oui ça peut ça aussi être un paramétrage géré depuis hobo, ça permettra à authentic de le connaitre également.
Mis à jour par Thomas Noël il y a presque 4 ans
Frédéric Péters a écrit :
Mais j'aurais aimé y mettre la liste des domaines connus mais à ne pas suggérer, comme noté dans #44663#note-8.
Mais oui ça peut ça aussi être un paramétrage géré depuis hobo, ça permettra à authentic de le connaitre également.
J'ai écrit ce ticket un peu sous le coup de la fatigue.
En fait je trouve ça très très bien que les paramétrages soient accessibles dans les backoffices. Mais effectivement, pour ceux qui se trouvent pilotés à distance, on devrait les afficher en read-only, avec, éventuellement, un hint qui explique qu'ils sont pilotés depuis "tel endroit".
Pour wcs, on pourrait faire un inventaire de ce qui est piloté par hobo_deploy (check_hobos.py), et quand une option est définie dans le hobo.json, la rendre read-only avec un message explicite "configuré via Hobo[https://hobo...]"
Genre
# wcs/qommon/admin/emails.py ... from wcs.ctl.check_hobos import is_hobo_setting ... if is_hobo_setting('emails', 'from'): form.add(StringWidget, 'from', title = _('Email Sender'), readonly='readonly', hint='from hobo', required = True, value = emails.get('from')) else: form.add(StringWidget, 'from', title = _('Email Sender'), required = True, value = emails.get('from'))
def is_hobo_setting(category, variable): variables = de la lecture du hobo.json ... if category == 'emails': if variable == 'from': return variables.get('default_from_email') is not None if variable == 'footer': return variables.get('email_signature') is not None ...
Bon... c'est un peu lourd et fastidieux, mais ça nous aiderait pas mal pour le support, je trouve. J'ai pas d'idée plus légère, j'ai juste l'objectif.
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/844
- Titre : WIP: settings: mark options set by hobo as readonly (#44756)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/844/files
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution proposée à Solution validée
Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/844
- Titre : settings: mark options set by hobo as readonly (#44756)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/844/files
Mis à jour par Transition automatique il y a 5 mois
- Statut changé de Résolu (à déployer) à Solution déployée
settings: allow not exposing smtp options (#44756)