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

[template] fix - change default page home, footer and header #404

Merged
merged 2 commits into from
May 28, 2024

Conversation

Louifine
Copy link
Collaborator

Il s'agit de :

  • Une correction de bug
  • Une nouvelle fonctionnalité programmée
  • Une nouvelle fonctionnalité spontanée

Décrivez vos changements

Cette PR est liée à la PR : Fix/header uv

Modification du template par défaut de la page d'accueil, du footer et du header des plateformes générées par recoco.
Le header ne contient que les logos essentiels et communs aux différentes plateformes.
Les blocks de contenu de la homepage sont retreints au minimum commun et les autres sont commentés pour être facilement réutilisés.
Le footer contient maintenant les pages légales (avec une adaptation minimale pour chaque plateforme).

Lien ticket project github

Checklist d'acceptation de revue de code

  • Aucun texte ne fait référence à du vocabulaire spécifique d'un métier
  • Mon code est auto-documenté ou la documentation a été mise à jour/créée
  • La gestion du multi-portail a été prise en compte
  • L'accessibilité a été prise en compte
  • Pre-commit est configuré et a été lancé
  • Des tests couvrant le code changé ont été ajoutés/modifiés
  • L'ensemble des tests front et back sont au vert

Demandes

  • Je souhaite un déploiement en préproduction

Cette PR modifiant la génération par défaut de la page d'accueil, du footer et du header des plateformes, une preprod permettrait d'être sûr qu'aucun site n'est "cassé".

Copy link

gitguardian bot commented May 21, 2024

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
4760140 Triggered Generic Password e81e752 frontend_tests/cypress/support/commands.js View secret
4760140 Triggered Generic Password e81e752 frontend_tests/cypress/support/commands.js View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Revoke and rotate the secret.

  2. If possible, rewrite git history with git commit --amend and git push --force.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@glibersat glibersat added this to the Release 2.29 milestone May 21, 2024
@glibersat glibersat merged commit 5fa26c0 into betagouv:develop May 28, 2024
3 of 4 checks passed
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.

None yet

2 participants