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

Show post meta fields in revisions panel #6735

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

SantosGuillamot
Copy link

@SantosGuillamot SantosGuillamot commented Jun 5, 2024

Trac ticket:

I can see that there was a similar closed ticket for this and a related pull request.

I just left a comment in the trac ticket to discuss how to proceed here.

@adamsilverstein @Mamaduka As you discussed this in the past, it would be great to hear your opinion on the best path forward.

What?

Basically, this pull request shows the changes made to the post meta fields that have revisions enabled in the revisions UI.

Why?

For 6.6, it is planned to allow the editing of custom fields when a block is connected to them. With that, I believe it is important to show the relevant changes in the revisions.

How?

It iterates through the wp_post_revision_meta_keys, and it creates a new row in the diff for each of the meta fields that have changed.

I excluded the footnotes because it has its own section right now, but I am not sure if it should be unified in the proposed "Post Meta".

Demo

This is a quick demo of how it works after the changes in this pull request:

Post.meta.revisions.with.key.demo.mp4

This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

Copy link

github-actions bot commented Jun 5, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props santosguillamot, mamaduka, gziolo.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link

github-actions bot commented Jun 5, 2024

Test using WordPress Playground

The changes in this pull request can previewed and tested using a WordPress Playground instance.

WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

Some things to be aware of

  • The Plugin and Theme Directories cannot be accessed within Playground.
  • All changes will be lost when closing a tab with a Playground instance.
  • All changes will be lost when refreshing the page.
  • A fresh instance is created each time the link below is clicked.
  • Every time this pull request is updated, a new ZIP file containing all changes is created. If changes are not reflected in the Playground instance,
    it's possible that the most recent build failed, or has not completed. Check the list of workflow runs to be sure.

For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation.

Test this pull request with WordPress Playground.

@Mamaduka
Copy link
Member

Sorry for the late reply, @SantosGuillamot! This got lost somewhere in my notifications pile 😓

I think @adamsilverstein's previous decision still holds true. Developers registering post meta/block blindings are better positioned to decide what to display on the revisions screen.

Reconsidering whether we should add this automatically - developers would be better suited to know what they want to display on this screen, core should only handle fields we add directly (eg. footnotes).

@gziolo
Copy link
Member

gziolo commented Jul 31, 2024

Reconsidering whether we should add this automatically - developers would be better suited to know what they want to display on this screen, core should only handle fields we add directly (eg. footnotes).

Slightly related. That also nicely ties into the previous discussion on whether to share the feedback for users in the UI. In case of Footnotes it was treated as an internal change that never should be surfaced to the users, while for other custom post meta there might be sometimes temptation to highlight that to users.

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.

3 participants