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

fix(glossary): correction des   sur la reconnaissance des mots #1511

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

maxgfr
Copy link
Member

@maxgfr maxgfr commented Dec 3, 2024

fix #1509

Copy link

sonarcloud bot commented Dec 3, 2024

Copy link

github-actions bot commented Dec 3, 2024

🎉 Deployment for commit 03a8589 :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/export:sha-03a8589b5146da90c2057d2a680eaf654e99bd5c
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/frontend:sha-03a8589b5146da90c2057d2a680eaf654e99bd5c
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/hasura:sha-03a8589b5146da90c2057d2a680eaf654e99bd5c
Debug

Comment on lines +546 to +551
test("should work with a composed name in tag with a non breaking space", async () => {
const htmlContent = "<p>Une zone commerciale&nbsp;</p>";
expect(await addGlossaryContent(glossaryData, htmlContent)).toEqual(
`<p>Une <webcomponent-tooltip content="Definition%20de%20la%20zone%20commerciale">zone commerciale</webcomponent-tooltip>&nbsp;</p>`
);
});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top le test 👍

@maxgfr maxgfr merged commit a1ceb18 into master Dec 9, 2024
47 of 48 checks passed
@maxgfr maxgfr deleted the maxgfr/glossary branch December 9, 2024 09:58
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.

Correction du Glossaire
4 participants