Skip to content

MR-29: Copy update to gutenberg visibility settings#21

Open
av3nger wants to merge 1 commit into
mainfrom
fix/MR-29-copy-updates
Open

MR-29: Copy update to gutenberg visibility settings#21
av3nger wants to merge 1 commit into
mainfrom
fix/MR-29-copy-updates

Conversation

@av3nger

@av3nger av3nger commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Update the wording of the visibility settings on the just-released gutenberg update to match what we have in the ad suppression visibility settings as well (which matches the metabox, etc). That way all of the language is consistent.


Note

Low Risk
String-only i18n label changes in the block editor with no access-control logic modified.

Overview
Updates Gutenberg block sidebar Memberful visibility strings so they match the metabox and ad suppression settings—no behavior or attribute changes.

The inspector panel title changes from Memberful Visibility to Memberful: Restrict Access. Dropdown labels are clarified: logged_in is now All members (active, inactive, or free) (was “All logged-in members”), and specific is Members with an active subscription (was “Specific membership plan”).

Reviewed by Cursor Bugbot for commit 9abe189. Bugbot is set up for automated code reviews on this repo. Configure here.

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: TheCodeCompany/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: ce0bf25a-5d2f-48f7-9216-d6ccb4037df9

📥 Commits

Reviewing files that changed from the base of the PR and between 26e44f1 and 9abe189.

📒 Files selected for processing (1)
  • wordpress/wp-content/plugins/memberful-wp/js/src/blocks/extensions/block-visibility/index.js

📝 Walkthrough

Summary by CodeRabbit

  • Improvements
    • Updated block visibility option labels in the Memberful plugin for improved clarity: "All members (active, inactive, or free)" and "Members with an active subscription"
    • Renamed the editor sidebar section title to "Memberful: Restrict Access" for better context

Walkthrough

Three UI label strings are updated in the block visibility extension: the SelectControl option label for logged_in changes to "All members (active, inactive, or free)", the specific option label changes to "Members with an active subscription", and the PanelBody sidebar title changes from "Memberful Visibility" to "Memberful: Restrict Access".

Changes

Block Visibility UI Label Updates

Layer / File(s) Summary
SelectControl labels and PanelBody title
wordpress/wp-content/plugins/memberful-wp/js/src/blocks/extensions/block-visibility/index.js
logged_in option label updated to "All members (active, inactive, or free)", specific option label updated to "Members with an active subscription", and PanelBody title changed from "Memberful Visibility" to "Memberful: Restrict Access".

Poem

Three little strings swapped out with care,
"Restrict Access" now hangs in the air. 🔒
Members active, inactive, or free —
The labels now clearer for all to see.
A tiny diff, but oh what a spree! 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating copy in Gutenberg visibility settings to match existing consistency standards.
Description check ✅ Passed The description clearly explains the purpose of the changes: standardizing visibility settings language across Gutenberg, ad suppression, and metabox components.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/MR-29-copy-updates

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

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

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9abe189. Configure here.

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