feat(dsfr): migration de la page plan du site #6097

m-maillot · 2024-09-05T06:55:23Z

No description provided.

github-advanced-security · 2024-09-05T07:09:31Z

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

m-maillot · 2024-09-05T06:56:04Z

packages/code-du-travail-frontend/cypress/integration/light/plan-du-site/plan-du-site.spec.ts

@@ -0,0 +1,16 @@
+describe("Plan du site", () => {
+  it("je vois le plan du site", () => {


Le test ne fonctionne actuellement pas : codegouvfr/react-dsfr#293

J'ai fix cette nuit 😅
Il suffit de mettre a jour react-dsfr a la dernière version.

CF: codegouvfr/react-dsfr#293 (comment)

m-maillot · 2024-09-05T06:56:36Z

packages/code-du-travail-frontend/package.json

@@ -114,7 +114,7 @@
    "stylelint-processor-styled-components": "^1.10.0",
    "supertest": "^6.3.3",
    "testing-library-selector": "0.3.1",
-    "typescript": "^5.0.4",
+    "typescript": "^5.5.4",


La version 5.5 permet de faire, enfin, des filters undefined sans hack :)

data.filter((item) => item !== undefined) // Renvoie bien un type sans undefined :)

Nice, ça serait intéressant d'upgrade partout en derniere version de TS, sur le package modele aussi

J'ai mis à jour partout :) J'étais obligé car si je reste en ancienne version sur les autres package, lerna récupère la plus vieille et ça ne marche pas bien...

m-maillot · 2024-09-05T06:58:28Z

packages/code-du-travail-frontend/pages/convention-collective/index.tsx

@@ -81,7 +81,15 @@ export default Page;

 export async function getStaticProps() {
  try {
-    let data = await getAllAgreements();
+    let data = await getAllAgreements([


les méthodes API getAll prend maintenant en paramètre les colonnes à récupérer (injecté ensuite dans le source). Cela permet d'optimiser pour chaque type de besoin.
Dans le plan du site, je n'ai besoin que du title / slug / idcc.

C'est bien mieux comme ça, de gérer directement au niveau du compoosant de savoir les items qu'on recupere

Par contre faudra bien definir une fonciton par page et pas d'avoir une fonction globale

C'est à dire ? Là j'ai une fonction getAll par type de contenu. On l'utilise sur le plan du site et sur la page listant les contenus s'il y en a une.

Pour les pages, on utilise la méthode générique getItem. Sur celle ci, j'aurai bien aimé aussi la typer plus fortement. C'est de ça dont tu parles ?

packages/code-du-travail-frontend/pages/integration/[slug].tsx

m-maillot · 2024-09-05T07:01:51Z

packages/code-du-travail-frontend/src/api/modules/agreements/service.ts

+export const getAllAgreements = async <K extends keyof ElasticAgreement>(
+  fields: K[],
+  sortBy?: K
+): Promise<Pick<ElasticAgreement, K>[]> => {


Voici la signature qui permet de passer les champs à récupérer et qui valide par la même occasion que les champs sont bien présent dans l'elastic (via le type ElasticAgreement qui est défini dans l'admin).

Ici j'ai un deuxième champ pour sort les résultats selon un champ. Je suis obligé de la passé en paramètre car je ne sais pas dans la méthode si le champ pour filtrer à été choisie. Après là je vois que l'on peut filtrer sur un champs que l'on a pas mis dans fields... A revoir je pense.

packages/code-du-travail-frontend/src/api/modules/sitemap/service.ts

m-maillot · 2024-09-05T07:14:35Z

...code-du-travail-frontend/src/api/modules/themes/__tests__/__snapshots__/service.test.ts.snap

-}
-`;
-
-exports[`Themes getAllThemesAndSubThemes 1`] = `


J'ai retiré cette méthode car on n'en a pas besoin pour le plan du site en fait. Par défaut le getRootThemes retourne les thèmes avec leurs children. Et dans leur children, on a toutes les infos nécessaires pour construire le plan du site :)

packages/code-du-travail-frontend/src/api/modules/themes/types.ts

m-maillot · 2024-09-05T07:16:34Z

packages/code-du-travail-frontend/src/api/modules/tools/queries.ts

+            term: {
+              displayTool: true,
+            },


Avant on faisait le filtre sur la méthode. Là je l'ai mis directement dans la requête.

m-maillot · 2024-09-05T07:17:19Z

packages/code-du-travail-frontend/src/modules/plan-du-site/index.tsx

+      <SiteMapSection
+        title={"Boîte à outils"}
+        url={"/outils"}
+        detail={{
+          source: SOURCES.TOOLS,
+          docs: tools,
+        }}
+      />


Comme j'ai les mêmes types, je peux avoir un composant commun.

maxgfr

Bien joué !! J'adore le fait de passer un array pour recuperer les champs qu'on souhaite, j'aurai juste paramètrer ça en poussant plus loin en le passant au niveau de la query elasticsearch

maxgfr · 2024-09-10T15:50:22Z

packages/code-du-travail-frontend/app/plan-du-site/page.tsx

+export const metadata: Metadata = {
+  title: "Plan du site - Code du travail numérique",
+  description: "Plan du site du Code du travail numérique",
+};


je le note pour pas oublier à gérer ça une fois qu'on merge la pr de caro

maxgfr · 2024-09-10T15:51:01Z

packages/code-du-travail-frontend/app/plan-du-site/page.tsx

+const getSiteMap = async () => {
+  return getSitemapData();
+};


De mon côté, j'avais fait du React.cache, mais je pense apres qu'il faut qu'on harmonise tout

Pourquoi pas utiliser le unstable_cache même si le nom fait peur

Le soucis du cache est qu'on ne le maitrise pas encore. On avait parlé de ne pas l'utiliser pour le moment et de voir pour l'invalider. Dans l'idée c'est bien d'utiliser unstable_cache mais avant on ne devait pas faire des tests avec les tags ?

maxgfr · 2024-09-10T15:52:09Z

packages/code-du-travail-frontend/cypress/integration/light/plan-du-site/plan-du-site.spec.ts

+  it("je vois le plan du site", () => {
+    cy.visit("/");
+    cy.get("a").contains("Plan du site").click();
+    cy.contains("h1", "Plan du site");


là faudra pas oublier d'utiliser l'accessibilité une fois qu'on merge la pr de caro

J'espère que ça fonctionnera avec les deux h1 :/

maxgfr · 2024-09-10T15:52:37Z

packages/code-du-travail-frontend/package.json

@@ -114,7 +114,7 @@
    "stylelint-processor-styled-components": "^1.10.0",
    "supertest": "^6.3.3",
    "testing-library-selector": "0.3.1",
-    "typescript": "^5.0.4",
+    "typescript": "^5.5.4",


Nice, ça serait intéressant d'upgrade partout en derniere version de TS, sur le package modele aussi

maxgfr · 2024-09-10T15:52:48Z

packages/code-du-travail-frontend/pages/convention-collective/index.tsx

@@ -81,7 +81,15 @@ export default Page;

 export async function getStaticProps() {
  try {
-    let data = await getAllAgreements();
+    let data = await getAllAgreements([


maxgfr · 2024-09-10T15:53:39Z

packages/code-du-travail-frontend/pages/convention-collective/index.tsx

@@ -81,7 +81,15 @@ export default Page;

 export async function getStaticProps() {
  try {
-    let data = await getAllAgreements();
+    let data = await getAllAgreements([


C'est bien mieux comme ça, de gérer directement au niveau du compoosant de savoir les items qu'on recupere

Par contre faudra bien definir une fonciton par page et pas d'avoir une fonction globale

maxgfr · 2024-09-10T15:54:11Z

packages/code-du-travail-frontend/src/api/modules/agreements/service.ts

+export const getAllAgreements = async <K extends keyof ElasticAgreement>(
+  fields: K[],
+  sortBy?: K
+): Promise<Pick<ElasticAgreement, K>[]> => {


maxgfr · 2024-09-10T15:55:23Z

packages/code-du-travail-frontend/src/api/modules/agreements/service.ts

+export const getAllAgreements = async <K extends keyof ElasticAgreement>(
+  fields: K[],
+  sortBy?: K
+): Promise<Pick<ElasticAgreement, K>[]> => {
  const body = getAllAgreementsWithContributions();


ah je pensais qu'on allait optimiser en passant l'arrant au niveau de la requete elasticsearch aussi.... t'en penses quoi ?

Je peux aussi le passer là. Dans l'idée, je passe directement dans la méthode search les _source pour éviter de trop complexifier le code. De plus, j'ai un soucis avec nos méthodes qui génère la requête ES car elles ne sont pas typées actuellement et je n'ai pas trouvé comment les typer...

maxgfr · 2024-09-10T15:58:56Z

packages/code-du-travail-frontend/src/modules/plan-du-site/index.tsx

+    {url ? (
+      <Link href={url}>
+        <span className={fr.cx("fr-text--lead", "fr-text--bold")}>{title}</span>
+      </Link>
+    ) : (
+      <span className={fr.cx("fr-text--lead", "fr-text--bold", "fr-mb-3w")}>
+        {title}
+      </span>
+    )}


On en a discuté aujourd'hui, mais c'est une vrai question. Laissons nous le frontend gérer cela ? Est-ce qu'il faudrait pas faire plutôt une approche de ce style :

{url && title && ( <Link href={url}> <span className={fr.cx("fr-text--lead", "fr-text--bold")}>{title}</span> </Link> ) } {title &&( <span className={fr.cx("fr-text--lead", "fr-text--bold")}>{title}</span> ) } }

Afin de rendre le composant le plus dumb possible

Pas compris la différence dans l'approche à la l'écriture ? D'ailleurs le title ne peut être null ici.

maxgfr · 2024-09-10T15:59:46Z

packages/code-du-travail-frontend/src/modules/plan-du-site/index.tsx

est-ce que ça vaudrait defaire un test unitaire sur le composant aussi ?

Oui bien sûr !

socket-security · 2024-09-11T08:12:24Z

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/[email protected]	environment	`0`	872 kB	mweststrate

View full report↗︎

github-actions · 2024-09-11T09:35:37Z

🎉 Deployment for commit 39009c2 :

Ingresses

🚀 https://code-du-travail-numerique-feat-page-sitemap-4zodrmjd.ovh.fabrique.social.gouv.fr/

Docker images

📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/code-du-travail-numerique/app:sha-39009c2d2f8ce4bd37170d1834da5075420e95db

Debug

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * fix: crash lors de la sélection d'une suggestion (#6202) * fix: crash lors de la sélection d'une suggestion * Update packages/code-du-travail-frontend/src/modules/utils/events.ts Co-authored-by: Caroline <[email protected]> --------- Co-authored-by: Caroline <[email protected]> * fix: crash lors de la sélection d'une suggestion (#6209) * fix: crash lors de la sélection d'une suggestion * Update packages/code-du-travail-frontend/src/modules/utils/events.ts Co-authored-by: Caroline <[email protected]> --------- Co-authored-by: Martial Maillot <[email protected]> * fix(suggestion): style + move feedback into RichComponent (#6210) * fix: format --------- Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: Victor Zeinstra <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * feat: implémentation simulateur brut net * chore: fix ts * chore: snap * chore: clean * chore: clean * chore: clean * chore: review * feat: update responsive * Revert "Merge branch 'dev' into simulateur-brut-net-dsfr" This reverts commit 0748928, reversing changes made to cd15bc7. * chore: clean * feat: chore clean * chore: clean * chore: remove undesired files * chore: remove undesired files * chore: clean * fix: meta description share * feat: switch simu brut net couleur par defaut * fix: ts * fix: bug load --------- Co-authored-by: maxgfr <[email protected]> Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * feat: implémentation simulateur brut net * chore: fix ts * chore: snap * chore: clean * chore: clean * chore: clean * chore: review * feat: update responsive * Revert "Merge branch 'dev' into simulateur-brut-net-dsfr" This reverts commit 0748928, reversing changes made to cd15bc7. * chore: clean * feat: chore clean * chore: clean * chore: remove undesired files * chore: remove undesired files * chore: clean * feat: implementation page convention collective * feat: update button link * fix: meta description share * feat: switch simu brut net couleur par defaut * feat: ajout de l'icone * feat: update svg * fix: ts * fix: bug load * chore: review * fix: mobile display * chore: review * chore: review * chore: review * chore review * fix: TU * chore: review * fix: validate html * chore: review * chore: clean --------- Co-authored-by: maxgfr <[email protected]> Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * feat: implémentation simulateur brut net * chore: fix ts * chore: snap * chore: clean * chore: clean * chore: clean * chore: review * feat: update responsive * Revert "Merge branch 'dev' into simulateur-brut-net-dsfr" This reverts commit 0748928, reversing changes made to cd15bc7. * chore: clean * feat: chore clean * chore: clean * chore: remove undesired files * chore: remove undesired files * chore: clean * feat: implementation page convention collective * feat: update button link * fix: meta description share * feat: switch simu brut net couleur par defaut * feat: ajout de l'icone * feat: update svg * fix: ts * fix: bug load * chore: review * fix: mobile display * chore: review * chore: review * chore: review * chore review * fix: TU * chore: review * fix: validate html * chore: review * chore: clean * chore: test e2e * fix: crypress * chore: cypress * fix: test e2e click header --------- Co-authored-by: maxgfr <[email protected]> Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * fix: css * fix: css * fix: css * fix: css * fix: css * fix: css * fix: merge date * fix: snapshot * fix: types * fix: tests * fix: tests * fix: tests * fix: tests * fix: lighthouse * fix: lighthouse * fix: bug * fix: bug * fix: UI feedbacks * fix: types * fix: don't modify old code * fix: image places * fix: quality issues * fix: issues * fix: espace manquant dans le titre * fix: espace manquant dans le h1 * Update packages/code-du-travail-frontend/src/modules/home/Components/HomeSearch.tsx Co-authored-by: Caroline <[email protected]> * Update packages/code-du-travail-frontend/src/modules/home/Components/HomeSearch.tsx Co-authored-by: Caroline <[email protected]> * fix: feedbacks * fix: feedbacks * fix: feedbacks * fix: html errors * try to add scroll to e2e * fix: merge conflicts * fix home * fix: use dblclick * fix: use dblclick * fix: use dblclick * fix: cypress issues * fix: cypress issues * fix: cypress issues * fix: cypress issues * fix: cypress issues * fix: cypress issues --------- Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: Victor Zeinstra <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

* fix: dsfr * fix: dsfr * fix: dsfr * fix: tests * fix: build * fix: build * fix: build * feat(dsfr): ajout du footer (#6079) * feat: add code * fix: tests * fix: tests * fix: tests * fix: tests * fix: tests * fix: dernier titi * fix: autoclick * fix: bug * feat(css): ajout de la lib `panda-css` (zero-runtime) (#6085) * fix: pandacss * fix: build * fix: build * feat(dsfr): ajout de la page stats (#6090) * fix: pandacss * fix: build * fix: build * fix(stats): add page * fix: stats * fix: tests * fix: tests * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix(recherche): remonter les pré-qualifiés dans la recherche (#6082) * chore(release): version 4.151.1 * fix(csp): remove reporting on sentry (#6092) * fix: config * fix: config * fix: config * empty * fix: config * empty * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * Update packages/code-du-travail-frontend/src/modules/mentions-legales/index.tsx Co-authored-by: Martial Maillot <[email protected]> * fix: config * chore(dsfr): mise à jour de la version DSFR * fix(dsfr): ignore les exceptions d'hydratation * feat(tests): ajout du module de testing (#6096) * fix: tests * fix: tests * fix: command * fix: merge date * fix: tests * fix: tests * fix: tests * fix: tests * fix(dsfr): ignore les exceptions d'hydratation * fix: readme * feat(DSFR): migration de la page article du code du travail (#6099) * feat: 6093 dsfr page politique de confidentialit (#6094) * feat: implémentation page politique confidentialité dsfr * chore: clean * fix: iframe dark mode * chore: refacto + e2e test * feat: convert a to Link * chore: review * chore: rename test --------- Co-authored-by: victor <[email protected]> * feat(dsfr): ajout des liens d'évitement (#6120) * feat(dsfr): migration de la page plan du site (#6097) * feat(dsfr): mise à jour des snapshots * feat(dsfr): mise à jour des snapshots * chore(dsfr): mise à jour de la lib DSFR * feat(dsfr): ajout du composant "Avez-vous trouvé une réponse à votre question" (#6121) * fix: merge date * fix: satisfaction * fix: retours preavis * fix: retours preavis * fix: tests * move feedback component to the page articleCodeDuTravail.tsx * clean-up css * fix: feedback * fix: feedback * fix: tests * fix: tests * feat(dsfr): création du composant "Besoin de plus d'information" (#6135) * fix: ui * fix: composant --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * fix: retours * fix: tests * feat: nouvelle API pour les articles du code du travail (#6132) Co-authored-by: carolineBda <[email protected]> * fix(spec) : fix de la spec article-code-du-travail.spec.ts * fix(dsfr): ajout de la config pour supprimer le `insafe-inline` des `scripts` dans les `csp` (#6151) * feat(dsfr): ajout de matomo pour tracker les events (#6157) * fix: matomo * fix: matomo --------- Co-authored-by: Martial Maillot <[email protected]> * fix(feedback): ajout d'une logique de caractères restants (#6156) * fix: tests * feat: limiter à 500 caractères la saisie * feat: limiter à 500 caractères la saisie --------- Co-authored-by: Martial Maillot <[email protected]> * feat(dsfr): ajout du nouveau logo (#6159) * feat(dsfr): ajout des pages d'erreurs (404 + 500) (#6146) * fix: pages * fix: 404 * fix: tests * fix: tests * fix: tests * fix: lint * test error * fix errors * Fix spec * add button to test error page * feat: force error 500 for testing purpose * feat: revert errors --------- Co-authored-by: carolineBda <[email protected]> Co-authored-by: Martial Maillot <[email protected]> * feat: séparation des anciennes API et des nouvelles (#6183) * fix(article code du travail): retrait du tag Code du travail (#6182) * feat(config): correction de `husky`, ajout de `prettier` pour le formattage et de `lint-staged` pour runner le formattage au `precommit` (#6192) * fix: prettier * fix: prettier * fix: readme * fix: readme * fix: bug * fix: prettier * fix: prettier * fix: tests * fix: tests * fix: prettier * fix: prettier * fix: branch * merge dev * feat: implémentation simulateur brut net * chore: fix ts * chore: snap * chore: clean * chore: clean * chore: clean * chore: review * feat: update responsive * Revert "Merge branch 'dev' into simulateur-brut-net-dsfr" This reverts commit 0748928, reversing changes made to cd15bc7. * chore: clean * feat: chore clean * chore: clean * chore: remove undesired files * chore: remove undesired files * chore: clean * feat: implementation page convention collective * feat: update button link * fix: meta description share * feat: switch simu brut net couleur par defaut * feat: ajout de l'icone * feat: update svg * fix: ts * fix: bug load * chore: review * fix: mobile display * chore: review * chore: review * chore: review * feat: implémentation trouver cc * feat: implementation trouver CC * chore: common style * feat: add error alert entreprise search * feat: disable card * feat: ajout TU + fix * chore: clean * fix: e2e tests * chore: clean * fix: widget * fix: error * feat: info bulle * fix: widget * fix: home fetch agreement * feat: add postmessage searchCC * fix: home cc * feat: change contenu lié search cc * fix: related items * fix: message erreur lors de la recherche * fix: error message * fix: relatedItems * chore: add TU * refactor: move files to folders * fix: build * chore: review * chore: clean * fix: ajout des canonicals * fix: input info * fix: canonicals * feat: ajout d'un spinner * fix: tests * chore: review * feat: add spinner to enterprise search * fix: load on select item autocomplete * fix: bug widget selection entreprise * fix: disabled rule * chore: review * chore: review * chore: review * chore: review * chore: review * chore: review * chore: review * fix: garder la recherche entrerprise au click precedent * chore: review * chore: review * fix: TU + some styles * fix: e2e * refactor * fix: TU * fix: TU * chore: review * chore: review * chore: review * chore: review postalcode * chore: review * chore: review * fix: cypress test * chore: add widgetMode TU * chore: fix test * fix: cypress test * fix: cypress test * chore: cy * chore: fix cy * refactor * refactor * chore: review * chore: clean * chore: review * fix: largeur champs recherche entreprise * feat: ajout d'un message d'erreur pour les codes naf * feat: update design * chore: clean * chore: design review * chore: review * chore: test noindex * chore: nbsp * chore: review * chore: review * chore: fix test e2e --------- Co-authored-by: maxgfr <[email protected]> Co-authored-by: Caroline <[email protected]> Co-authored-by: Martial Maillot <[email protected]> Co-authored-by: Social Groovy Bot <[email protected]> Co-authored-by: victor <[email protected]> Co-authored-by: carolineBda <[email protected]>

m-maillot commented Sep 5, 2024

View reviewed changes

m-maillot force-pushed the feat/page-sitemap branch from 83418bb to 4595e34 Compare September 10, 2024 12:54

m-maillot temporarily deployed to review-auto September 10, 2024 12:54 — with GitHub Actions Inactive

m-maillot marked this pull request as ready for review September 10, 2024 12:55

m-maillot requested review from Viczei, carolineBda and maxgfr September 10, 2024 12:55

maxgfr reviewed Sep 10, 2024

View reviewed changes

Viczei approved these changes Sep 10, 2024

View reviewed changes

m-maillot force-pushed the feat/page-sitemap branch from 4595e34 to ccbbb57 Compare September 11, 2024 08:11

m-maillot temporarily deployed to review-auto September 11, 2024 08:11 — with GitHub Actions Inactive

m-maillot force-pushed the feat/page-sitemap branch from ccbbb57 to 39009c2 Compare September 11, 2024 09:23

m-maillot temporarily deployed to review-auto September 11, 2024 09:23 — with GitHub Actions Inactive

feat(dsfr): migration de la page plan du site

9e5823a

m-maillot force-pushed the feat/page-sitemap branch from 39009c2 to 9e5823a Compare September 16, 2024 12:24

m-maillot had a problem deploying to review-auto September 16, 2024 12:24 — with GitHub Actions Failure

m-maillot merged commit 75b9216 into maxgfr/dsfr-header Sep 16, 2024
5 of 6 checks passed

m-maillot deleted the feat/page-sitemap branch September 16, 2024 12:25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(dsfr): migration de la page plan du site #6097

feat(dsfr): migration de la page plan du site #6097

m-maillot commented Sep 5, 2024

github-advanced-security bot commented Sep 5, 2024

m-maillot Sep 5, 2024

garronej Sep 5, 2024

garronej Sep 5, 2024

m-maillot Sep 5, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

m-maillot Sep 5, 2024

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

m-maillot Sep 5, 2024

maxgfr Sep 10, 2024

m-maillot Sep 5, 2024

m-maillot Sep 5, 2024

m-maillot Sep 5, 2024

maxgfr left a comment

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

maxgfr Sep 10, 2024

m-maillot Sep 11, 2024

socket-security bot commented Sep 11, 2024 •

edited

Loading

github-actions bot commented Sep 11, 2024

		@@ -0,0 +1,16 @@
		describe("Plan du site", () => {
		it("je vois le plan du site", () => {

feat(dsfr): migration de la page plan du site #6097

feat(dsfr): migration de la page plan du site #6097

Conversation

m-maillot commented Sep 5, 2024

github-advanced-security bot commented Sep 5, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

maxgfr left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

socket-security bot commented Sep 11, 2024 • edited Loading

github-actions bot commented Sep 11, 2024

socket-security bot commented Sep 11, 2024 •

edited

Loading