Skip to content

Conversation

@ilicfilip
Copy link
Collaborator

Currently we add PP Sidebar to all edit screens and for all users, which is wrong.

This PR adds a check so it is only loaded for users who have edit_others_posts cap.

Also it adds a check so sidebar is only added to the Page edit screen, but.. I am not sure about it. We have Blog page type:
Screenshot 2025-12-17 at 13 09 30

And I am not sure if this should be assigned to all Blog posts.. or we should remove it?

On settings page we had: Home, About, Contact and FAQ

cc @jdevalk

@github-actions
Copy link
Contributor

Test on Playground
Test this pull request on the Playground
or download the zip

@github-actions
Copy link
Contributor

✅ Code Coverage Report

Metric Value
Total Coverage 31.48% 📉
Base Coverage 31.49%
Difference 📉 -0.01%

⚠️ Coverage below recommended 40% threshold

📊 File-level Coverage Changes (1 files)

📉 Coverage Decreased

Class Before After Change
Progress_Planner\Admin\Editor 3.85% 3.57% -0.28%
ℹ️ About this report
  • All tests run in a single job with Xdebug coverage
  • Security tests excluded from coverage to prevent output issues
  • Coverage calculated from line coverage percentages

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.

2 participants