feat(glossary): from export to publish #1424
Conversation
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
| @@ -27,13 +27,13 @@ export type ContributionElasticDocumentLightRelatedContent = Omit< | |||
| "linkedContent" | |||
| > & { | |||
| linkedContent: ContributionLinkedContent[]; | |||
| contentWithGlossary?: string; | |||
There was a problem hiding this comment.
Il est optional pour le cas des fiches SP ?
There was a problem hiding this comment.
Je crois j'avais fait ça par simplicité pour éviter de modifier le type en le destructurant. Mais oui, ce champ n'existe pas en théorie. Mais ça demandait pas mal de refacto au niveau des types....
| delete document.intro; | ||
| delete document.introWithGlossary; |
There was a problem hiding this comment.
je ne suis pas fan des delete, on ne peux pas passer par une destructuration ?
There was a problem hiding this comment.
Ouais j'avais essayé, mais c'était moins lisible je trouve
| } | ||
| } | ||
|
|
||
| private async updateGlossaryForEditorialContents(glossary: Glossary) { |
There was a problem hiding this comment.
On n'a pas le glossaire ailleurs aussi comme sur les fiches MT ?
There was a problem hiding this comment.
Ailleurs, c'est à dire ?
De mon analyse, pour notre contenu, c'était les contributions et les pages infos. Et en externe juste les fiches MT
|
|
🎉 Deployment for commit 03b007b : Docker images
|
fix #1407