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

#5754 - Allow CGU URL and Legal Notice URL to be configured in .env file #5755

Conversation

dzc34
Copy link
Collaborator

@dzc34 dzc34 commented Nov 20, 2020

Fixed #5754 "ETQ Ops, je souhaite changer les URLs des Mentions légales et des CGU " / @adullact

Actuellement

L'URL des liens Mentions légales et CGU sont déclarées dans le fichier config/initializers/urls.rb.
Ce n'est pas configurable sans modifier le code source.

Lorsqu'on installe sa propre instance DS (par exemple celle de l'Adullact) sans modifier le code source, les liens en bas de pages
Mentions légales et CGU amènent l'utilisateur sur les documents légaux relatifs au site web demarches-simplifiees.fr

<li class="footer-link-cgu">
    <a href="https://doc.demarches-simplifiees.fr/cgu" rel="noopener noreferrer"
      target="_blank">CGU</a>
</li>
<li class="footer-link-mentions-legales">
    <a href="https://doc.demarches-simplifiees.fr/mentions-legales" rel="noopener noreferrer"
       target="_blank">Mentions  légales</a>
</li>

Comportement attendu

Rendre configurable les URL des liens Mentions légales et CGU.

<li class="footer-link-cgu">
    <a href="https://example.com/cgu-service-ds" rel="noopener noreferrer"
      target="_blank">CGU</a>
</li>
<li class="footer-link-mentions-legales">
    <a href="https://example.com/mentions-legales-service-ds" rel="noopener noreferrer"
       target="_blank">Mentions  légales</a>
</li>

Implémentation

Modification du fichier config/initializers/urls.rb, pour utiliser des variables d'environnements optionnelles CGU_URL et MENTIONS_LEGALES_URL et les documenter dans le fichier env.example.optional.

-CGU_URL = [DOC_URL, "cgu"].join("/")
-MENTIONS_LEGALES_URL = [DOC_URL, "mentions-legales"].join("/")
+CGU_URL = ENV.fetch("CGU_URL", [DOC_URL, "cgu"].join("/"))
+MENTIONS_LEGALES_URL = ENV.fetch("MENTIONS_LEGALES_URL", [DOC_URL, "mentions-legales"].join("/"))

trackingAdullactContrib

@dzc34 dzc34 changed the title allow CGU URL and Legal Notice URL to be configured in .env file #5754 - Allow CGU URL and Legal Notice URL to be configured in .env file Nov 20, 2020
@LeSim LeSim merged commit 6ee2bec into demarches-simplifiees:dev Nov 23, 2020
@dzc34 dzc34 deleted the 5754_allow_CGU_LEGAL_URLS_to_be_configured_in_ENV_file branch November 26, 2020 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ETQ Ops, je souhaite changer les URLs des Mentions légales et des CGU
2 participants