feat: migration vers le nouveau format MT #423

Viczei · 2024-09-25T12:49:53Z

4 tests ont disparu car je n'ai pas trouvé ces cas dans les nouvelles fiches.

J'ai ajouté de nouveaux tests pour gérer les nouveaux cas.

socket-security · 2024-10-16T07:29:26Z

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

Package	New capabilities	Transitives	Size	Publisher
npm/@types/[email protected]	None	`0`	21.9 kB	types
npm/@types/[email protected]	None	`0`	13.5 kB	types
npm/[email protected]	None	`0`	413 kB	feedic
npm/[email protected]	None	`0`	702 kB	feedic

View full report↗︎

m-maillot · 2024-10-16T08:46:01Z

src/fetch-data/__tests__/parseDom.test.ts

-    expect(parsed.sections[0].anchor).toBe("");
-  });
-
-  test("should parse HTML with 4 sections with id", () => {


C'est le cas nominal. On le test plus bas sur une autre page.

m-maillot · 2024-10-16T08:47:02Z

src/fetch-data/__tests__/parseDom.test.ts

    );
    expect(parsed).toMatchSnapshot();
  });

-  test("should work with picture more complex", () => {


On ne récupère plus l'image dans l'intro.

m-maillot · 2024-10-16T08:47:17Z

src/fetch-data/__tests__/parseDom.test.ts

  });

-  test("should keep image within picture if it has a source", () => {


On ne récupère plus l'image dans l'intro. De plus la page utilisée n'est pas disponible sur notre site.

m-maillot · 2024-10-16T09:14:33Z

src/fetch-data/__tests__/parseDom.test.ts

  });

-  test("should work with error", () => {


Cet article posé peut être des soucis avant mais la nouvelle version ne pose pas de soucis. Pas besoin de ce test.

m-maillot · 2024-10-16T09:15:18Z

src/fetch-data/__tests__/parseDom.test.ts

  });

-  test("should work with script intro", () => {


Cet article n'est plus récupéré et je n'ai pas vu de cas avec un script dans l'intro. D'ailleurs la nouvelle version n'a plus de script dans l'intro.

maxgfr

GG pour le refacto, c'est top !!

Si c'est un projet qu'on souhaite pereniser. Serait-il pas pertinent de le passer en TS aussi ?

maxgfr · 2024-10-16T18:04:46Z

.github/workflows/release.yml


-      #- name: Check Refs
-      #  run: yarn checkRefs


On l'avait enlevé car ça buggait avant, c'est ça ?

C'était pour désactiver la mise à jour des données si on lançait le job à la main. Il n'y avait pas de bug spécifique à cette étape.

maxgfr · 2024-10-16T18:07:53Z

src/fetch-data/__tests__/parseDom.test.ts

+    expect(parsed.sections[9].html).toBe(
+      '<h3>Indemnité spécifique de rupture conventionnelle</h3><p> À l\'occasion de la rupture conventionnelle de son contrat de travail, le salarié doit percevoir une <strong>«</strong> <strong>indemnité spécifique de rupture conventionnelle</strong> <strong>»</strong> dont le <strong>montant</strong>, négocié avec l’employeur et précisé dans la convention de rupture, <strong>ne peut être inférieur au montant de</strong> l\'<a href="https://travail-emploi.gouv.fr/lindemnite-legale-de-licenciement" data-entity-type="node" data-entity-uuid="9f586260-18e9-4665-afb9-63ab7263130a" data-entity-substitution="canonical" title="L\'indemnité légale de licenciement" target="_blank" rel="nofollow, noopener">indemnité légale de licenciement</a>. </p><div class="fr-highlight"> <p> Comme le précise la Cour de cassation dans un arrêt du 11 mai 2022 <a href="https://www.courdecassation.fr/decision/627b53774d359c057dd01ceb" target="_blank" rel=" noopener external" title="auquel on se reportera - nouvelle fenêtre">auquel on se reportera</a>, « la créance d\'indemnité de rupture conventionnelle, si elle n\'est exigible qu\'à la date fixée par la rupture, naît dès l\'homologation de la convention ». Ainsi, dans l’hypothèse où le salarié décède après l’homologation de la convention de rupture, l’employeur doit verser l’indemnité de rupture aux héritiers du salarié. </p> </div><p> <strong>Aucune condition d’ancienneté n’est requise pour avoir droit à cette indemnité</strong> ; toutefois, si le salarié partie à la rupture conventionnelle a moins de 8 mois d’ancienneté dans l’entreprise (ancienneté requise pour avoir droit à l’indemnité légale de licenciement), l’indemnité spécifique de rupture conventionnelle lui est due au prorata du nombre de mois de présence. </p><div class="fr-highlight"> <p> Aux termes de l\'avenant n° 4 du 18 mai 2009 à l\'<strong>accord national interprofessionnel (ANI)</strong> sur la modernisation du marché du travail du 11 janvier 2008, l\'indemnité spécifique de rupture à verser au salarié dont le CDI a été rompu dans le cadre d\'une rupture conventionnelle de l\'article L. 1237-11 du Code du travail, doit être au moins égale à l\'indemnité conventionnelle de licenciement, dès lors que cette dernière s\'avère plus favorable, pour le salarié, que l\'indemnité légale. </p> <p> Cet avenant ne s\'est d\'abord imposé qu\'aux employeurs adhérents du MEDEF, de la CGPME ou de l\'UPA, et ce au titre des conventions de rupture conventionnelle conclues depuis le 18 mai 2009. Son extension par l\'arrêté du 26 novembre 2009 (JO du 27 novembre) le rend désormais obligatoire à l\'égard de tous les employeurs entrant dans le champ d\'application de l\'ANI du 11 janvier 2008 (c\'est-à-dire les employeurs du secteur privé, exception faite, notamment, des professions libérales, du secteur associatif et des employeurs de salariés agricoles), et ce au titre des conventions de rupture conclues depuis le 28 novembre 2009. Les conventions conclues précédemment ne sont pas remises en cause. </p> </div><p> La rupture conventionnelle est également ouverte aux <strong>salariés du particulier employeur, titulaires d’un contrat de travail à durée indéterminée</strong>. Dans ce cas, l’indemnité de rupture conventionnelle doit être au moins égale à l’indemnité légale de licenciement. </p><p> <em>À contrario</em>, <strong>la rupture conventionnelle n’est pas ouverte aux assistants maternels</strong>, conformément à la volonté des partenaires sociaux confirmée par la nouvelle convention collective nationale des particuliers employeurs et de l\'emploi à domicile du 15 mars 2021. </p><h3>Régime fiscal et social</h3><p> <strong>N\'est pas soumise à l\'impôt sur le revenu</strong>, la fraction de l\'indemnité spécifique versée à l\'occasion de la rupture conventionnelle du contrat de travail d\'un salarié <strong>lorsqu\'il n\'est pas en droit de bénéficier d\'une pension de retraite d\'un régime légalement obligatoire</strong>, qui n\'excède pas : </p><ul> <li> <strong>Soit deux fois le montant de la rémunération annuelle brute perçue par le salarié</strong> au cours de l\'année civile précédant la rupture de son contrat de travail, ou 50 % du montant de l\'indemnité si ce seuil est supérieur, dans la limite de 6 fois le plafond annuel de la sécurité sociale (PASS) en vigueur à la date de versement des indemnités ; </li> <li> <strong>Soit le montant de l\'indemnité de licenciement</strong> prévue par la convention collective de branche, par l\'accord professionnel ou interprofessionnel ou, à défaut, par la loi. </li> </ul><p> <strong>Lorsque le salarié est susceptible de bénéficier d’une pension de retraite d’un régime légalement obligatoire</strong>, l’indemnité de rupture conventionnelle est <strong>assujettie à l’impôt sur le revenu dès le premier euro</strong>. </p><p> Sur le régime social de cette indemnité (cotisations sociales, CSG, CRDS, contribution patronale), on se reportera aux précisions figurant au <a href="https://boss.gouv.fr/portail/accueil/indemnites-de-rupture.html#titre-chapitre-4---rupture-conventionn-section-2--regime-social-de-lind" target="_blank" rel=" noopener external" title="Bulletin officiel de la sécurité sociale (BOSS) - nouvelle fenêtre">Bulletin officiel de la sécurité sociale (BOSS)</a>. </p><h3> Autres indemnités et documents à remettre au salarié </h3><p> <strong>S\'il quitte l\'entreprise avant d\'avoir pu prendre la totalité des congés payés qu\'il avait acquis</strong>, le salarié a droit à une <strong>indemnité compensatrice</strong> de <a href="https://travail-emploi.gouv.fr/les-conges-payes" data-entity-type="node" data-entity-uuid="17264e9b-01ff-4c2a-8469-409865c087d9" data-entity-substitution="canonical" title="Les congés payés" target="_blank" rel="nofollow, noopener">congés payés</a>, ainsi qu\'à l\'ensemble des éléments de rémunération dus par l\'employeur à la date de la rupture du contrat de travail. </p><div class="fr-highlight"> <p> Au moment du départ de l\'entreprise, doit remettre au salarié (ou, selon le cas, tenir à sa disposition) un certain nombre <a href="https://travail-emploi.gouv.fr/les-documents-remis-aux-salaries-lors-de-la-rupture-du-contrat-de-travail" data-entity-type="node" data-entity-uuid="b0de4179-13c5-43ca-81f3-9579f607b3e5" data-entity-substitution="canonical" title="Les documents remis aux salariés lors de la rupture du contrat de travail" target="_blank" rel="nofollow, noopener">de documents</a> : certificat de travail, solde de tout compte, attestation pour faire valoir ses droits aux allocations chômage (dite « attestation France Travail » ou « attestation Pôle emploi » – France Travail, nouvel opérateur du service public de l’emploi, remplace Pôle emploi depuis le <a href="https://www.francetravail.fr/actualites/france-travail/pole-emploi-france-travail.html" target="_blank" rel=" noopener external" title="1er janvier 2024 - nouvelle fenêtre">1<sup>er</sup> janvier 2024</a>), etc. </p> </div>'


Perso, je preferais le separer du code car, ça rend le fichier de test ultra long comme ça

J'ai remplacé par des toMatchSnapshot() pour plus de lisibilité.

maxgfr · 2024-10-16T18:08:52Z

src/fetch-data/index.js

-    http2: true,
-    retry: 3,
-  });
-  const { fiches: feed } = JSON.parse(response.body);
-  const localJson = fs.readFileSync(
+export async function fetchFeed() {
+  const localJsonData = fs.readFileSync(
    path.join(__dirname, "../../local.data.json"),
    "utf8"
  );
-  const { fiches: localFeed } = JSON.parse(localJson);
-  return [...feed, ...localFeed];
+  return JSON.parse(localJsonData).fiches;


Oui, maintenant la liste n'est plus dynamique mais statique. Est-ce un comportement qu'on souhaite ?

Oui, en attendant la remise en place de l'API par l'équipe des fiches MT

maxgfr · 2024-10-16T18:10:38Z

src/fetch-data/parseDom.js

Est-ce qu'on a encore besoin des fonctions qu'on a mis plus haut ? unwrapEmail, getCleanSrc ?

On les garde par sécurité. Je n'ai pas validé si elles étaient encore utiles.

# [4.701.0](v4.700.0...v4.701.0) (2024-10-31) ### Features * **data:** 20241031_0953 update ([85724f0](85724f0)) * migration vers le nouveau format MT ([#423](#423)) ([044f1a8](044f1a8))

github-actions · 2024-10-31T09:53:54Z

🎉 This PR is included in version 4.701.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

feat: migration vers le nouveau format MT

5bc9d05

Viczei requested review from m-maillot, carolineBda and maxgfr September 25, 2024 12:49

Viczei self-assigned this Sep 25, 2024

Viczei removed the request for review from maxgfr September 25, 2024 12:50

victor and others added 3 commits September 26, 2024 16:02

chore: update tests

2f07056

fix: video page section display

e4338d7

test: mise à jour des données avec les fiches au format DSFR

68c8baf

m-maillot force-pushed the feat/migration-MT-update branch 2 times, most recently from 66bcffa to 96acc62 Compare October 16, 2024 08:40

m-maillot requested a review from maxgfr October 16, 2024 08:41

m-maillot force-pushed the feat/migration-MT-update branch from 96acc62 to d3232ee Compare October 16, 2024 08:43

victor added 3 commits October 16, 2024 10:43

feat: migration vers le nouveau format MT

9757a52

chore: update tests

18256ce

fix: video page section display

19bc033

m-maillot force-pushed the feat/migration-MT-update branch 2 times, most recently from d6abacc to 378fbb8 Compare October 16, 2024 09:13

test: mise à jour des données avec les fiches au format DSFR

b7eb204

m-maillot force-pushed the feat/migration-MT-update branch from 378fbb8 to b7eb204 Compare October 16, 2024 09:18

m-maillot reviewed Oct 16, 2024

View reviewed changes

maxgfr approved these changes Oct 16, 2024

View reviewed changes

Merge branch 'feat/migration-MT-update' into test-mt-dsfr

3cda874

m-maillot force-pushed the feat/migration-MT-update branch from 3924ce2 to 1245b67 Compare October 22, 2024 06:48

test: migrate to match snapshot

1dc801a

m-maillot force-pushed the feat/migration-MT-update branch from 1245b67 to 1dc801a Compare October 22, 2024 06:57

fix: improve highlight parsing

621e5bb

m-maillot approved these changes Oct 22, 2024

View reviewed changes

m-maillot force-pushed the feat/migration-MT-update branch from 09d1456 to 621e5bb Compare October 22, 2024 11:43

m-maillot merged commit 044f1a8 into master Oct 31, 2024
3 checks passed

m-maillot deleted the feat/migration-MT-update branch October 31, 2024 09:45

github-actions bot added the released label Oct 31, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: migration vers le nouveau format MT #423

feat: migration vers le nouveau format MT #423

Viczei commented Sep 25, 2024 •

edited by m-maillot

Loading

socket-security bot commented Oct 16, 2024 •

edited

Loading

m-maillot Oct 16, 2024

m-maillot Oct 16, 2024

m-maillot Oct 16, 2024

m-maillot Oct 16, 2024

m-maillot Oct 16, 2024

maxgfr left a comment

maxgfr Oct 16, 2024

m-maillot Oct 22, 2024

maxgfr Oct 16, 2024

m-maillot Oct 22, 2024

maxgfr Oct 16, 2024

m-maillot Oct 22, 2024

maxgfr Oct 16, 2024

m-maillot Oct 22, 2024

github-actions bot commented Oct 31, 2024

		});

		test("should keep image within picture if it has a source", () => {

		expect(parsed.sections[9].html).toBe(
		'<h3>Indemnité spécifique de rupture conventionnelle</h3><p> À l\'occasion de la rupture conventionnelle de son contrat de travail, le salarié doit percevoir une <strong>«</strong> <strong>indemnité spécifique de rupture conventionnelle</strong> <strong>»</strong> dont le <strong>montant</strong>, négocié avec l’employeur et précisé dans la convention de rupture, <strong>ne peut être inférieur au montant de</strong> l\'<a href="https://travail-emploi.gouv.fr/lindemnite-legale-de-licenciement" data-entity-type="node" data-entity-uuid="9f586260-18e9-4665-afb9-63ab7263130a" data-entity-substitution="canonical" title="L\'indemnité légale de licenciement" target="_blank" rel="nofollow, noopener">indemnité légale de licenciement</a>. </p><div class="fr-highlight"> <p> Comme le précise la Cour de cassation dans un arrêt du 11 mai 2022 <a href="https://www.courdecassation.fr/decision/627b53774d359c057dd01ceb" target="_blank" rel=" noopener external" title="auquel on se reportera - nouvelle fenêtre">auquel on se reportera</a>, « la créance d\'indemnité de rupture conventionnelle, si elle n\'est exigible qu\'à la date fixée par la rupture, naît dès l\'homologation de la convention ». Ainsi, dans l’hypothèse où le salarié décède après l’homologation de la convention de rupture, l’employeur doit verser l’indemnité de rupture aux héritiers du salarié. </p> </div><p> <strong>Aucune condition d’ancienneté n’est requise pour avoir droit à cette indemnité</strong> ; toutefois, si le salarié partie à la rupture conventionnelle a moins de 8 mois d’ancienneté dans l’entreprise (ancienneté requise pour avoir droit à l’indemnité légale de licenciement), l’indemnité spécifique de rupture conventionnelle lui est due au prorata du nombre de mois de présence. </p><div class="fr-highlight"> <p> Aux termes de l\'avenant n° 4 du 18 mai 2009 à l\'<strong>accord national interprofessionnel (ANI)</strong> sur la modernisation du marché du travail du 11 janvier 2008, l\'indemnité spécifique de rupture à verser au salarié dont le CDI a été rompu dans le cadre d\'une rupture conventionnelle de l\'article L. 1237-11 du Code du travail, doit être au moins égale à l\'indemnité conventionnelle de licenciement, dès lors que cette dernière s\'avère plus favorable, pour le salarié, que l\'indemnité légale. </p> <p> Cet avenant ne s\'est d\'abord imposé qu\'aux employeurs adhérents du MEDEF, de la CGPME ou de l\'UPA, et ce au titre des conventions de rupture conventionnelle conclues depuis le 18 mai 2009. Son extension par l\'arrêté du 26 novembre 2009 (JO du 27 novembre) le rend désormais obligatoire à l\'égard de tous les employeurs entrant dans le champ d\'application de l\'ANI du 11 janvier 2008 (c\'est-à-dire les employeurs du secteur privé, exception faite, notamment, des professions libérales, du secteur associatif et des employeurs de salariés agricoles), et ce au titre des conventions de rupture conclues depuis le 28 novembre 2009. Les conventions conclues précédemment ne sont pas remises en cause. </p> </div><p> La rupture conventionnelle est également ouverte aux <strong>salariés du particulier employeur, titulaires d’un contrat de travail à durée indéterminée</strong>. Dans ce cas, l’indemnité de rupture conventionnelle doit être au moins égale à l’indemnité légale de licenciement. </p><p> <em>À contrario</em>, <strong>la rupture conventionnelle n’est pas ouverte aux assistants maternels</strong>, conformément à la volonté des partenaires sociaux confirmée par la nouvelle convention collective nationale des particuliers employeurs et de l\'emploi à domicile du 15 mars 2021. </p><h3>Régime fiscal et social</h3><p> <strong>N\'est pas soumise à l\'impôt sur le revenu</strong>, la fraction de l\'indemnité spécifique versée à l\'occasion de la rupture conventionnelle du contrat de travail d\'un salarié <strong>lorsqu\'il n\'est pas en droit de bénéficier d\'une pension de retraite d\'un régime légalement obligatoire</strong>, qui n\'excède pas : </p><ul> <li> <strong>Soit deux fois le montant de la rémunération annuelle brute perçue par le salarié</strong> au cours de l\'année civile précédant la rupture de son contrat de travail, ou 50 % du montant de l\'indemnité si ce seuil est supérieur, dans la limite de 6 fois le plafond annuel de la sécurité sociale (PASS) en vigueur à la date de versement des indemnités ; </li> <li> <strong>Soit le montant de l\'indemnité de licenciement</strong> prévue par la convention collective de branche, par l\'accord professionnel ou interprofessionnel ou, à défaut, par la loi. </li> </ul><p> <strong>Lorsque le salarié est susceptible de bénéficier d’une pension de retraite d’un régime légalement obligatoire</strong>, l’indemnité de rupture conventionnelle est <strong>assujettie à l’impôt sur le revenu dès le premier euro</strong>. </p><p> Sur le régime social de cette indemnité (cotisations sociales, CSG, CRDS, contribution patronale), on se reportera aux précisions figurant au <a href="https://boss.gouv.fr/portail/accueil/indemnites-de-rupture.html#titre-chapitre-4---rupture-conventionn-section-2--regime-social-de-lind" target="_blank" rel=" noopener external" title="Bulletin officiel de la sécurité sociale (BOSS) - nouvelle fenêtre">Bulletin officiel de la sécurité sociale (BOSS)</a>. </p><h3> Autres indemnités et documents à remettre au salarié </h3><p> <strong>S\'il quitte l\'entreprise avant d\'avoir pu prendre la totalité des congés payés qu\'il avait acquis</strong>, le salarié a droit à une <strong>indemnité compensatrice</strong> de <a href="https://travail-emploi.gouv.fr/les-conges-payes" data-entity-type="node" data-entity-uuid="17264e9b-01ff-4c2a-8469-409865c087d9" data-entity-substitution="canonical" title="Les congés payés" target="_blank" rel="nofollow, noopener">congés payés</a>, ainsi qu\'à l\'ensemble des éléments de rémunération dus par l\'employeur à la date de la rupture du contrat de travail. </p><div class="fr-highlight"> <p> Au moment du départ de l\'entreprise, doit remettre au salarié (ou, selon le cas, tenir à sa disposition) un certain nombre <a href="https://travail-emploi.gouv.fr/les-documents-remis-aux-salaries-lors-de-la-rupture-du-contrat-de-travail" data-entity-type="node" data-entity-uuid="b0de4179-13c5-43ca-81f3-9579f607b3e5" data-entity-substitution="canonical" title="Les documents remis aux salariés lors de la rupture du contrat de travail" target="_blank" rel="nofollow, noopener">de documents</a> : certificat de travail, solde de tout compte, attestation pour faire valoir ses droits aux allocations chômage (dite « attestation France Travail » ou « attestation Pôle emploi » – France Travail, nouvel opérateur du service public de l’emploi, remplace Pôle emploi depuis le <a href="https://www.francetravail.fr/actualites/france-travail/pole-emploi-france-travail.html" target="_blank" rel=" noopener external" title="1er janvier 2024 - nouvelle fenêtre">1<sup>er</sup> janvier 2024</a>), etc. </p> </div>'

feat: migration vers le nouveau format MT #423

feat: migration vers le nouveau format MT #423

Conversation

Viczei commented Sep 25, 2024 • edited by m-maillot Loading

socket-security bot commented Oct 16, 2024 • edited Loading

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

github-actions bot commented Oct 31, 2024

Viczei commented Sep 25, 2024 •

edited by m-maillot

Loading

socket-security bot commented Oct 16, 2024 •

edited

Loading