-
Notifications
You must be signed in to change notification settings - Fork 77
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
Post meta: Update meta keys to be private #1090
base: trunk
Are you sure you want to change the base?
Conversation
@pfefferle I remember you mentioning certain post meta keys that should be private, which ones were those? |
It was more about if there are some that should not be private. Like for example the visibility one. So that classic editor users can add that through the meta forms for example. |
WordPress convention states that meta keys prefixed with an underscore are considered private and hidden from the custom fields UI. Since our meta keys are for internal plugin use only, they should follow this convention. This change helps prevent accidental modification of ActivityPub meta data through WordPress's custom fields interface while maintaining the expected pattern for plugin developers. See: https://developer.wordpress.org/plugins/metadata/managing-post-metadata/#hidden-custom-fields
Needs to be editable in editor
edfd9b5
to
cbf88af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
WordPress convention states that meta keys prefixed with an underscore are considered private and hidden from the custom fields UI. Since our meta keys are for internal plugin use only, they should follow this convention.
This change helps prevent accidental modification of ActivityPub meta data through WordPress's custom fields interface while maintaining the expected pattern for plugin developers.
See: https://developer.wordpress.org/plugins/metadata/managing-post-metadata/#hidden-custom-fields
Fixes #943
Proposed changes:
class-test-followers.php
to use the new prefixed meta keys (activitypub*)class-test-migration.php
to test the migration directly instead of throughmaybe_migrate()
activitypub_user_id
→_activitypub_user_id
)Other information:
Testing instructions: