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(plugin-seo): return empty if localized fields are only seo #30

Merged

Conversation

thompsonsj
Copy link
Owner

This is not how this should be done.

It is an interim fix while I figure out how to handle a scenario where there are no localized fields on a collection except the localized fields added by https://github.com/payloadcms/plugin-seo.

By default, payload-crowdin-sync detects localized fields on a collection or global, and then syncs with crowdIn automatically. If SEO is enabled on a collection or global without localized fields, it will start syncing with crowdIn for the meta fields only.

This makes the following scenario likely: #29.

To keep this default behaviour, I am implementing this measure. This should be removed once #29 is resolved because only localized SEO fields is a reasonable scenario (e.g. a gallery of images or a collection that contains fields unsupported by the plugin).

@thompsonsj thompsonsj merged commit 02145ac into main Jun 8, 2023
@thompsonsj thompsonsj mentioned this pull request Dec 15, 2023
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.

1 participant