Projet

Général

Profil

Development #44756

cacher backoffice/settings/emails/options

Ajouté par Thomas Noël il y a presque 4 ans. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
02 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

Révision aa716d19 (diff)
Ajouté par Frédéric Péters il y a 5 mois

settings: allow not exposing smtp options (#44756)

Révision d546f91f (diff)
Ajouté par Frédéric Péters il y a 5 mois

settings: mark options set by hobo as readonly (#44756)

Historique

#1

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.

#2

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.

#3

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 :

#4

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de En cours à Solution proposée
#5

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 :

#6

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 :

#7

Mis à jour par Transition automatique il y a 5 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Transition automatique il y a 3 mois

Automatic expiration

Formats disponibles : Atom PDF