Skip to content

Align Manage Pages documentation with current UI behavior - #100

Merged
bkatusic merged 1 commit into
mainfrom
docs-review/manage-pages
Jul 22, 2026
Merged

bkatusic merged 1 commit into
mainfrom
docs-review/manage-pages

Conversation

@bkatusic

@bkatusic bkatusic commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update page search documentation to reflect modified-date filtering and configurable result limits.
  • Align create, edit, trash, and permanent-deletion instructions with current MainWP Dashboard labels and behavior.
  • Clarify that page titles and content are optional when creating pages.

Testing

  • Not run (not requested)

Summary by CodeRabbit

  • Documentation
    • Updated Manage Pages guidance for date-range filtering and page result limits.
    • Clarified the Create New Page workflow and required versus optional fields.
    • Updated editing instructions to use Publish.
    • Clarified bulk deletion, permanent deletion availability, and irreversible deletion warnings.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Updated the Manage Pages documentation to reflect current search settings, page creation and publishing labels, field requirements, and bulk or permanent deletion behavior.

Changes

Manage Pages documentation

Layer / File(s) Summary
Search guidance
sites/content/manage-pages.mdx
Updates Date Range guidance to use last-modified dates and documents the Max pages to return setting.
Page creation and editing
sites/content/manage-pages.mdx
Updates the Create New Page navigation, field requirement indicators, and Publish instruction.
Page deletion behavior
sites/content/manage-pages.mdx
Clarifies bulk deletion wording, Trash-only permanent deletion actions, and irreversible bulk deletion behavior.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • mainwp/docs#85: Updates related page-publishing documentation and Manage Pages UI terminology.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the documentation update to match current UI behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs-review/manage-pages

Comment @coderabbitai help to get the list of available commands.

@mintlify

mintlify Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
mainwp 🟢 Ready View Preview Jul 22, 2026, 2:44 PM

@bkatusic
bkatusic merged commit 76b2aab into main Jul 22, 2026
4 of 5 checks passed
@bkatusic
bkatusic deleted the docs-review/manage-pages branch July 22, 2026 14:44

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@sites/content/manage-pages.mdx`:
- Line 79: Update the enclosing step title in the Manage Pages instructions from
“Navigate to Add New” to “Navigate to Create New Page” so it matches the current
UI label and the instruction text.
- Around line 181-185: Standardize the deletion action labels in the
page-management documentation: update the visible references around “Bulk
Actions” and the per-page action so “Move to trash” and “Delete permanently” use
consistent casing throughout the flow.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f6ea9ee-4fa7-4b23-ac3a-683c935f5448

📥 Commits

Reviewing files that changed from the base of the PR and between b5e2187 and d5ceb12.

📒 Files selected for processing (1)
  • sites/content/manage-pages.mdx

<Steps>
<Step title="Navigate to Add New">
Go to **MainWP > Sites > Content Operations > Manage Pages**, then click **Add New**.
Go to **MainWP > Sites > Content Operations > Manage Pages**, then click **Create New Page**.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Rename the step title to match the new UI label.

The instruction now tells users to click Create New Page, but the enclosing step title still says Navigate to Add New. Update the title so the navigation guidance is internally consistent.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@sites/content/manage-pages.mdx` at line 79, Update the enclosing step title
in the Manage Pages instructions from “Navigate to Add New” to “Navigate to
Create New Page” so it matches the current UI label and the instruction text.

Comment on lines +181 to +185
For multiple pages, select them using checkboxes and use **Bulk Actions > Move to trash**.

### Delete Permanently

WordPress only allows permanent deletion of pages already in Trash.
The per-page **Delete permanently** action appears only for pages in Trash.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Inspect the referenced section
git grep -n "Move to trash\|Delete permanently\|Delete Permanently\|Trash" -- sites/content/manage-pages.mdx sites || true

echo
echo "---- referenced lines ----"
sed -n '170,195p' sites/content/manage-pages.mdx

echo
echo "---- broader matches ----"
rg -n "Move to trash|Delete permanently|Delete Permanently|Trash" sites/content sites || true

Repository: mainwp/docs

Length of output: 7154


Standardize the page deletion labels
This section mixes Move to trash/Delete permanently with Move to Trash/Delete Permanently; use one casing consistently across the flow.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@sites/content/manage-pages.mdx` around lines 181 - 185, Standardize the
deletion action labels in the page-management documentation: update the visible
references around “Bulk Actions” and the per-page action so “Move to trash” and
“Delete permanently” use consistent casing throughout the flow.

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.

1 participant