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

Add graph api custom def (#98) #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tempSt69
Copy link
Contributor

@tempSt69 tempSt69 commented Dec 2, 2024

Close: #98

A prendre avec des pincettes car pourrait drastiquement augmenter le nombre de mots de tous les projets utilisants graphapi.

Je pense qu'on devrait au moins commencer avec les type "Product" car on a beaucoup de Shopify et je crois que tous les produits on un ld+json graph par défaut.

Exemple avec le shop de Vitality. On est en anglais mais le graphapi reste en français :
image

@wfulgenzi
Copy link
Contributor

wfulgenzi commented Dec 2, 2024

c'est cool !
Que penses-tu de rajouter une nouvelle condition, similaire à XML_ATTRIBUTE_VALUE pour le XMLResource.js.
Dans ce cas ça serait qqchose comme JSON_PATH_VALUE. La condition de type,

type: "JSON_PATH_VALUE",
payload: {
   path: [JSON path value],
   match: [same as other string matches for URL condition payloads]
}

This would allow us to test the JSON object itself for the presence of a certain key / value, e.g., in this case, @type: "Product", and in that case we could be confident that it's OK to try to translate all the different associated product terms.

OK pardon je n'avais pas compris que JSONPath permettait des conditions durant l'assignement, très stylé !

@tempSt69
Copy link
Contributor Author

tempSt69 commented Dec 5, 2024

Ca vous dit qu'on commence seulement avec le type product ? on voit pour le reste ensuite ?

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.

Add every SEO graph schema.org data to translation
2 participants