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(content-distribution): pages table column #230

Merged
merged 3 commits into from
Mar 17, 2025

Conversation

miguelpeixe
Copy link
Member

#228 failed to recognize that the manage_posts_columns covers the columns for posts and custom post types, but not pages.

This PR adds the Content Distribution column to the Pages table.

Testing

Visit the Pages admin page and confirm the content distribution column is present in the table.

@miguelpeixe miguelpeixe requested a review from a team as a code owner March 17, 2025 16:52
@miguelpeixe miguelpeixe self-assigned this Mar 17, 2025
@miguelpeixe miguelpeixe requested a review from naxoc March 17, 2025 16:52
Copy link
Member

@naxoc naxoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works! 👍

@miguelpeixe miguelpeixe merged commit fe5fa5c into trunk Mar 17, 2025
4 checks passed
@miguelpeixe miguelpeixe deleted the fix/content-distribution-pages-column branch March 17, 2025 16:58
Copy link

Hey @miguelpeixe, good job getting this PR merged! 🎉

Now, the needs-changelog label has been added to it.

Please check if this PR needs to be included in the "Upcoming Changes" and "Release Notes" doc. If it doesn't, simply remove the label.

If it does, please add an entry to our shared document, with screenshots and testing instructions if applicable, then remove the label.

Thank you! ❤️

matticbot pushed a commit that referenced this pull request Mar 17, 2025
# [2.8.0-alpha.3](v2.8.0-alpha.2...v2.8.0-alpha.3) (2025-03-17)

### Bug Fixes

* **content-distribution:** pages table column ([#230](#230)) ([fe5fa5c](fe5fa5c))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 2.8.0-alpha.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Mar 18, 2025
# [2.8.0](v2.7.0...v2.8.0) (2025-03-18)

### Bug Fixes

* **content-distribution:** pages table column ([#230](#230)) ([fe5fa5c](fe5fa5c))

### Features

* **content-distribution:** remove distribution on incoming post deletion ([#225](#225)) ([24a421a](24a421a))
* **content-distribution:** support page post type ([#228](#228)) ([1d21b4d](1d21b4d))
* **event-log:** migrate `data` db column to `longtext` ([#227](#227)) ([ef2353b](ef2353b))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 2.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants