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

Remove outdated properties in WPSEO_Meta for translations #21575

Closed
Tracked by #21550
vraja-pro opened this issue Aug 29, 2024 · 0 comments · Fixed by #21592
Closed
Tracked by #21550

Remove outdated properties in WPSEO_Meta for translations #21575

vraja-pro opened this issue Aug 29, 2024 · 0 comments · Fixed by #21592
Assignees

Comments

@vraja-pro
Copy link
Contributor

vraja-pro commented Aug 29, 2024

The $meta_fields properties are used in the Meta_Fields_Presenter to render those fields.
Currently the only properties that are being used to render the fields are the key.
For saving and sanitising the metadata the properties "default_value" is used but the default is always an empty string.
The sanitisation is using the options property.

@vraja-pro vraja-pro self-assigned this Aug 29, 2024
@vraja-pro vraja-pro changed the title Deprecate title and description in WPSEO_Meta since we are not adding translations on server side Remove outdated properties in WPSEO_Meta since we are not adding translations on server side Aug 29, 2024
@vraja-pro vraja-pro linked a pull request Aug 29, 2024 that will close this issue
18 tasks
@vraja-pro vraja-pro removed their assignment Aug 30, 2024
@vraja-pro vraja-pro changed the title Remove outdated properties in WPSEO_Meta since we are not adding translations on server side Remove outdated properties in WPSEO_Meta for translations Sep 3, 2024
@vraja-pro vraja-pro removed a link to a pull request Sep 3, 2024
18 tasks
@vraja-pro vraja-pro assigned igorschoester and unassigned vraja-pro Sep 3, 2024
@vraja-pro vraja-pro assigned igorschoester and unassigned vraja-pro Sep 4, 2024
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 a pull request may close this issue.

2 participants