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

Restaure les itemprop perdus #6672

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

philippemilink
Copy link
Member

  • le bloc article_schema n'est redéfinit nul part, donc autant le supprimer et utiliser directement sa valeur
  • personne ne définit de bloc article_centent_schema ou article_content_schema (sans la typo), donc suppression également

Ce commit restaure juste les attributs schema.org perdus avec c29c53f, mais il faudrait sans doute refaire une passe complète sur ces attributs ; ce sera pour une autre PR.

Fix #6604

Contrôle qualité

Aller sur la page publique d'un contenu et s'assurer que les attributs schema.org sont à nouveau présents. L'exemple donné dans le sujet rapportant le problème peut aider pour savoir ce qu'il faut chercher.

- le bloc article_schema n'est redéfinit nul part, donc autant le
  supprimer et utiliser directement sa valeur
- personne ne définit de bloc article_centent_schema ou
  article_content_schema (sans la typo), donc suppression également

Ce commit restaure juste les attributs schema.org perdus avec
c29c53f, mais il faudrait sans doute
refaire une passe complète sur ces attributs.

Fix zestedesavoir#6604
@coveralls
Copy link

coveralls commented Oct 20, 2024

Coverage Status

coverage: 89.187%. remained the same
when pulling bc0d733 on philippemilink:restore-itemprop
into 42b51ff on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

Comme tu dis, on corrige la régression, mais il faudrait effectivement faire une passe pour choisir de meilleurs attributs.

@Arnaud-D Arnaud-D enabled auto-merge (squash) October 22, 2024 19:26
@Arnaud-D Arnaud-D merged commit 1a6e95c into zestedesavoir:dev Oct 22, 2024
8 checks passed
@philippemilink philippemilink deleted the restore-itemprop branch October 22, 2024 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Régression sur les métadonnées Schema.org des contenus
3 participants