Skip to content

Restricted visibility settings in the member email editor#26692

Merged
9larsons merged 2 commits intomainfrom
restrict-viz
Mar 4, 2026
Merged

Restricted visibility settings in the member email editor#26692
9larsons merged 2 commits intomainfrom
restrict-viz

Conversation

@9larsons
Copy link
Copy Markdown
Contributor

@9larsons 9larsons commented Mar 4, 2026

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cea0b9d0-3f15-431f-a591-2ff6051d55a8

📥 Commits

Reviewing files that changed from the base of the PR and between 6181784 and a82a45e.

📒 Files selected for processing (1)
  • apps/admin-x-settings/src/components/settings/membership/member-emails/member-email-editor.tsx

Walkthrough

The pull request adds a visibilitySettings property with value 'none' to the cardConfig object in the member email editor component, alongside the existing image allowedWidths configuration. This augments the configuration passed to KoenigEditorBase without changing type imports, component interfaces, or external APIs. The change is minimal (net +2/−1 lines) and does not modify exported public APIs.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Restricted visibility settings in the member email editor' directly and specifically describes the main change of adding visibilitySettings: 'none' to the cardConfig in the member email editor.
Description check ✅ Passed The description references a Linear issue (NY-1124) related to the change, which provides context for the pull request purpose even though minimal details are given.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch restrict-viz

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.

@9larsons 9larsons enabled auto-merge (squash) March 4, 2026 14:24
@9larsons 9larsons merged commit 68f0990 into main Mar 4, 2026
30 checks passed
@9larsons 9larsons deleted the restrict-viz branch March 4, 2026 14:49
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