Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URL générée pour une application : organisme court #293

Closed
lecault opened this issue Mar 13, 2024 · 7 comments
Closed

URL générée pour une application : organisme court #293

lecault opened this issue Mar 13, 2024 · 7 comments
Labels
discussion to talk on a subject enhancement
Milestone

Comments

@lecault
Copy link
Collaborator

lecault commented Mar 13, 2024

Problématique

Avec le déploiement de mviewerstudio sur GéoBretagne, nous avons remarqué que si l'intitulé de l'organisme du partenaire est long, cela génère une URL à rallonge.

Exemples :
en preview : https://geobretagne.fr/mviewer/#store/agence_d_attractivite_et_de_developpement_de_redon_agglomeration/75ff09939d07/preview/541360d6-e117-11ee-bcde-005056909889
en prod : https://geobretagne.fr/mviewer/#prod/agence_d_attractivite_et_de_developpement_de_redon_agglomeration/test

Proposition de solution

Pouvoir choisir via un paramètre dans la conf json d'utiliser plutôt l'organisme court. Ce qui donnerai dans notre cas : https://geobretagne.fr/mviewer/#prod/redon_attractivite/test

@lecault lecault added enhancement discussion to talk on a subject labels Mar 13, 2024
@Gaetanbrl
Copy link
Member

A voir pour changer le champ utilisé :

orgname = request.headers.get("sec-orgname")

@lecault
Copy link
Collaborator Author

lecault commented Mar 26, 2024

C'est bien ici, j'ai remplacé sec-orgname par sec-org et ça fonctionne.

Maintenant ce que je cherche à faire c'est de passer cette valeur en paramètre.

J'ai ajouté dans le service :
Environment="TYPE_ORG=sec_org"

Et j'ai essayé comme ci-dessous de récupérer la variable dans le login_utils.py mais ça ne marche pas.

    typeorg = current_app.config["TYPE_ORG"]
    orgname = request.headers.get(typeorg)

@Gaetanbrl
Copy link
Member

Maintenant ce que je cherche à faire c'est de passer cette valeur en paramètre.

Pourquoi ne pas toujours prendre le nom court ?

@lecault
Copy link
Collaborator Author

lecault commented Mar 26, 2024

Je me dis que ça peut dépendre de la façon de saisir le nom.

On pourrait le mettre par défaut et mettre en commentaire la valeur du nom long si ça te semble pas pertinent de mettre un paramètre.

@Gaetanbrl
Copy link
Member

Je n'ai pas trop d'avis sur la question, le nom court est également utile pour la création des répertoires sur le serveur (mieux que le nom long, plus simple).
A la rigueur, on peut mettre un contrôle sur le nombre de caractère et afficher le nom long dans un cas et le nom court dans l'autre.
Si on ne met que le nom court ca me convient, je ne sais pas ce qu'en penserai les utilisateurs cela dit.

@lecault
Copy link
Collaborator Author

lecault commented Mar 26, 2024

Voici un exemple de ce que ça donne en nom normal :

image

Du coup, je penche pour mettre mon court par défaut. Je fais une PR

@Gaetanbrl
Copy link
Member

Du coup, je penche pour mettre mon court par défaut. Je fais une PR

👍

@lecault lecault closed this as completed Mar 26, 2024
@lecault lecault added this to the 4.0.2 milestone Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion to talk on a subject enhancement
Projects
None yet
Development

No branches or pull requests

2 participants