feat: migration vers le nouveau format MT #423
+136,248
−74,486
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
m-maillot
force-pushed
the
feat/migration-MT-update
branch
2 times, most recently
from
66bcffa
to
96acc62
Compare
m-maillot
force-pushed
the
feat/migration-MT-update
branch
from
96acc62
to
d3232ee
Compare
m-maillot
force-pushed
the
feat/migration-MT-update
branch
2 times, most recently
from
d6abacc
to
378fbb8
Compare
m-maillot
force-pushed
the
feat/migration-MT-update
branch
from
378fbb8
to
b7eb204
Compare
m-maillot
reviewed
Oct 16, 2024
| expect(parsed.sections[0].anchor).toBe(""); | ||
| }); | ||
|
|
||
| test("should parse HTML with 4 sections with id", () => { |
There was a problem hiding this comment.
C'est le cas nominal. On le test plus bas sur une autre page.
| ); | ||
| expect(parsed).toMatchSnapshot(); | ||
| }); | ||
|
|
||
| test("should work with picture more complex", () => { |
There was a problem hiding this comment.
On ne récupère plus l'image dans l'intro.
| }); | ||
|
|
||
| test("should keep image within picture if it has a source", () => { |
There was a problem hiding this comment.
On ne récupère plus l'image dans l'intro. De plus la page utilisée n'est pas disponible sur notre site.
| }); | ||
|
|
||
| test("should work with error", () => { |
There was a problem hiding this comment.
Cet article posé peut être des soucis avant mais la nouvelle version ne pose pas de soucis. Pas besoin de ce test.
| }); | ||
|
|
||
| test("should work with script intro", () => { |
There was a problem hiding this comment.
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
approved these changes
Oct 16, 2024
|
|
||
| #- name: Check Refs | ||
| # run: yarn checkRefs |
There was a problem hiding this comment.
On l'avait enlevé car ça buggait avant, c'est ça ?
There was a problem hiding this comment.
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.
Comment on lines
104
to
105
| 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>' |
There was a problem hiding this comment.
Perso, je preferais le separer du code car, ça rend le fichier de test ultra long comme ça
There was a problem hiding this comment.
J'ai remplacé par des toMatchSnapshot() pour plus de lisibilité.
Comment on lines
-15
to
+16
| 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; |
There was a problem hiding this comment.
Oui, maintenant la liste n'est plus dynamique mais statique. Est-ce un comportement qu'on souhaite ?
There was a problem hiding this comment.
Oui, en attendant la remise en place de l'API par l'équipe des fiches MT
src/fetch-data/parseDom.js
Outdated
There was a problem hiding this comment.
Est-ce qu'on a encore besoin des fonctions qu'on a mis plus haut ? unwrapEmail, getCleanSrc ?
There was a problem hiding this comment.
On les garde par sécurité. Je n'ai pas validé si elles étaient encore utiles.
m-maillot
force-pushed
the
feat/migration-MT-update
branch
from
3924ce2
to
1245b67
Compare
m-maillot
force-pushed
the
feat/migration-MT-update
branch
from
1245b67
to
1dc801a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.