Produits Entr'ouvert » Hobo

Il manque le templates/manager_home.html, non ?

À part ça, ma principale question : c'est obligatoire d'avoir fait un hobos.json et un hobo.json, plutôt que de garder la même URL ?

Mon idée c'était qu'un portail admin déployé simplement, sans multitenant, on pouvait y trouver un hobos.json, même path, que le programme agent appelant puisse fonctionner sans avoir à déployer un portail-admin en multi-tenant, ni créer un deuxième mode pour aller prendre le hobo.json.

Serghei Mihai a écrit :

Chargement des classes pour le mode multitenant au moment des appels aux fonctions multitenant

forms.py pourrait être utilisé plus tard pour d'autres forms : j'aurais fait un try/import dans le HoboForm, genre :

from django.forms import ModelForm
try:
    from tenant_schemas.utils import get_tenant_model
except ImportError:
    get_tenant_model = lambda: None

class HoboForm(ModelForm):
    class Meta:
        model = get_tenant_model()
        fields = ['schema_name', 'domain_url']

J'aime pinailler.

Je me suis fait la même remarque (peut-être que j'aime bien pinailler aussi).

Dans ce cas, peut-être qu'il sera plus approprié de separer les formulaires liés à la gestion des tenants dans un module à part.

Les patchs 0002 et 0004 devraient être éliminés, et je mettrais 0001 et 0003 en un unique patch.

J'obtiens des comportements bizarres dans le focus des zones de texte, sans doute parce que plusieurs éléments se trouvent avec le même id, id_domain_url.

Je mettrais le champ "domain url" avant le champ "schema name", et idéalement celui-ci ne serait pas obligatoire mais créé sur base du "domain url" (comme dans la commande create-tenant, if schema_name is None: schema_name = slugify(domain_url).

Ça ne se fait pas d'avoir "oui" ou "non" comme libellé de boutons, ceux-ci devraient être "Supprimer" et "Annuler". Idéalement il faudrait aussi inverser les boutons et qu'ils soient alignés à droite.

J'avais suggéré que 1 et 3 soient réunis, tu gardes le 1 identique, tu élimines le 3 en intégrant le patch de #5064 dans le nouveau 2. Franchement c'est difficile à suivre et je n'y pige pas le sens.

Soit trois patchs : 1) le 0001 actuel, 2) le 0002 limité à l'édition et suppression (c'est son titre, après tout) , 3) le fonctionnement quand tenant_schemas n'est pas installé.

Soit deux patchs : 1) le 0001 fusionné avec le fonctionnement quand tenant_schemas n'est pas installé, le 0002 pareillement limité à l'édition et suppresion.

Dans 0001-..., le commentaire sur "def hobos()" s'applique plutôt à "def hobo()".

Le placement du commentaire au bon endroit, je l'aurais plutôt vu directement correct dans le 0001; à part ça je dirais que c'est ok, tu peux faire cette modif et pousser les patchs (en y incluant le numéro du ticket) ?

Appliqué par commit 8ff21f95168f48f3f7c0beb0670024f0501d0827.