Skip to content

fix: harden account example return context and empty cards - #195

Draft
superdav42 wants to merge 1 commit into
mainfrom
fix/frontend-account-example-hardening
Draft

superdav42 wants to merge 1 commit into
mainfrom
fix/frontend-account-example-hardening

Conversation

@superdav42

@superdav42 superdav42 commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Harden the published account example after the requested code-quality review.

  • static/downloads/frontend-account/canvas.php: preserve the validated request path and query through sign-in and password-recovery links, including selected-site and membership context.
  • static/downloads/frontend-account/ums-account-example.php: after native blocks/shortcodes render, remove only completely empty example-owned card wrappers on the designated account canvas. Populated cards and non-example pages are left alone. Bump the example and stylesheet version to 1.0.1.
  • static/downloads/frontend-account/portal.css: remove content-hiding selectors tied to .wu-current-site, .wu-styling and .wu-invoices descendants. Navigation highlighting is unchanged.
  • Keep the seven-file ZIP, full inline source and DESIGN.md consistent. No generator, extra dependency or new test infrastructure.

Verification

  • PHP syntax checks pass.
  • Actual guest links retain the selected-site path and membership query, and a real login returns to that exact customer context.
  • Real shortcode and native-block landing/selected views have no empty cards or hidden populated cards; mobile has no horizontal overflow.
  • Existing WordPress content-filter path verified: whitespace-only cards removed, changed widget classes and input-only cards retained, external return URLs rejected.
  • The seven inline source blocks and ZIP entries match their downloadable files.
  • English production docs build passes without broken-link warnings. Independent review of actual desktop/mobile renders found no empty bordered placeholders or clipping in the supplied views.

Related plugin change

Ultimate-Multisite/ultimate-multisite#2086

The ownership regression test there now protects fixture setup inside its cleanup boundary, guards the temporary super-admin cleanup and restores the previous site selector. This docs PR contains no plugin test changes.


aidevops.sh v3.39.119 plugin for OpenCode

@superdav42 superdav42 added the origin:interactive Auto-created by pulse labelless backfill (t2112) label Nov 6, 2026
@coderabbitai

coderabbitai Bot commented Nov 6, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

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

Labels

origin:interactive Auto-created by pulse labelless backfill (t2112)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant