Development #11488
multitenant: exposer un tenant aux commandes django
0%
Description
Avec la commande
manage tenant_command <commande>
seul le schema de la base est modifié, la commande executée ne pas au courant dans quel tenant elle est executé.
Des fois il est utile que la commande puisse connaître son tenant(son hostname, par exemple).
Ça serait bien de passer l'objet tenant dans les arguments de la commande.
Fichiers
Demandes liées
Révisions associées
settings loaders: add get_new_time support to SiteBaseUrl loader (#11488)
Historique
Mis à jour par Serghei Mihai il y a presque 8 ans
- Fichier 0001-multitenant-pass-tenant-object-to-management-command.patch 0001-multitenant-pass-tenant-object-to-management-command.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 8 ans
je confirme, c'est connection.get_tenant() qu'il faut utiliser, toujours.
Mis à jour par Frédéric Péters il y a presque 8 ans
Mais surtout, tenant_command, c'est fait pour exécuter des commandes qui ne sont pas au courant qu'elles fonctionnent dans un environnement multitenants. Ton souhait, ce ne serait pas plutôt de connaître l'URL de base du site ?
Mis à jour par Frédéric Péters il y a presque 8 ans
Donc ce qu'il faut c'est s'accorder sur une variable (settings.SITE_BASE_URL ?) qui contiendrait l'url de base du site, et s'assurer qu'elle soit positionnée automatiquement aux bons endroits.
Mis à jour par Thomas Noël il y a presque 8 ans
Parce qu'on n'en a jamais eu besoin avant, ça m'interpelle : c'est pour quel usage ?
Mis à jour par Serghei Mihai il y a presque 8 ans
C'est pour Corbo, l'idée étant expliquée dans le ticket #10805: dans le rendu html des annonces construire les urls vers les statics
Mis à jour par Frédéric Péters il y a presque 8 ans
J'ai commenté là, comme quoi non, il vaut mieux ne pas faire de liens vers les images.
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
C'est déjà développé connection.get_tenant().get_base_url() ça gère https/pas https via le fichier unsecure dans le répertoire du tenant (par défaut c'est toujours https).
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
Pour le mail utiliser le tag <base> ne suffirait-il pas ?
PS: et non apparemment ce n'est pas une bonne idée, http://stackoverflow.com/questions/14611225/html-base-tag-in-email
Mis à jour par Frédéric Péters il y a presque 8 ans
- Fichier 0001-settings-loaders-add-SITE_BASE_URI-to-tenant-setting.patch 0001-settings-loaders-add-SITE_BASE_URI-to-tenant-setting.patch ajouté
J'écrivais :
Donc ce qu'il faut c'est s'accorder sur une variable (settings.SITE_BASE_URL ?) qui contiendrait l'url de base du site, et s'assurer qu'elle soit positionnée automatiquement aux bons endroits.
Mis à jour par Frédéric Péters il y a presque 8 ans
- Lié à Bug #12543: Erreur d'URL vers le désabonnement ajouté
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit fb9927bd0a0def5026336f8ecc11eb4c183b284f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Jul 12 13:45:15 2016 +0200
settings loaders: add SITE_BASE_URI to tenant settings (#11488)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Fichier 0001-settings-loaders-add-get_new_time-support-to-SiteBas.patch 0001-settings-loaders-add-get_new_time-support-to-SiteBas.patch ajouté
- Statut changé de Résolu (à déployer) à En cours
C'était passé en local dans des tests trop sommaires.
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit f34624a78e318cfbbf186abff475de718fcb46c8
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Jul 12 15:27:45 2016 +0200
settings loaders: add get_new_time support to SiteBaseUrl loader (#11488)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
settings loaders: add SITE_BASE_URI to tenant settings (#11488)