chore: rephrasing zone membership BED-7197

[LucasParraF]

Description

On explore page, when right clicking on a node and selecting Add to Tier Zero, the confirmation modal message was rephrase to keep cohesion with the terminology used on the app

Motivation and Context

Resolves BED-7197

How Has This Been Tested?

Tested locally

Screenshots (optional):

Types of changes

• Chore (a change that does not modify the application functionality)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

Summary by CodeRabbit

Release Notes

• Bug Fixes
  • Updated confirmation messages in asset group management dialogs to use consistent terminology throughout the application.

[coderabbitai]

📝 Walkthrough

Walkthrough

Two confirmation dialog messages in asset group menu item components are updated to use "zone membership" terminology instead of "group membership" when managing Tier Zero asset group operations. No functional or control flow changes are made.

Changes

Cohort / File(s)	Summary
Asset Group Menu Dialogs cmd/ui/src/views/Explore/ContextMenu/AssetGroupMenuItem.tsx, packages/javascript/bh-shared-ui/src/views/Explore/ContextMenu/AssetGroupMenuItemPrivilegeZonesEnabled.tsx	Updated confirmation dialog text to reference "zone membership" instead of "group membership" in two dialog instances for adding/removing nodes from asset groups in Tier Zero scenarios.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• BED-6709 chore: default tags glyph migration #2022 — Modifies the structure and imports of AssetGroupMenuItemPrivilegeZonesEnabled.tsx, the same module affected by this text update.

Suggested labels

user interface

Suggested reviewers

• mistahj67
• urangel
• Holocraft

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The pull request description includes required sections and addresses the core change, but testing details are vague and some documentation claims lack specificity.	Provide specific details about local testing (e.g., which browsers, environments, or test steps were performed) and clarify which documentation was reviewed to ensure comprehensive validation.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: rephrasing 'group membership' to 'zone membership' in confirmation dialogs across two files, with reference to the Jira ticket BED-7197.
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

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch BED-7197-User-confirms-analysis-to-update-zone-membership

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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