feat: group publishing by label - #72
Merged
Merged
Conversation
BIA3IA
commented
Sep 4, 2026
BIA3IA
commented
Contributor
- Added PublishTagGroupsDialog to manage the visibility and untagging of groups with a specific tag.
- Created TagGroupsPage to display groups associated with a flat tag, allowing for search and management.
- Introduced useLabelGroupRows hook to streamline the retrieval of group rows based on labels.
- Enhanced CreateEditGroupDialog to include an option to hide groups until published.
- Updated CombinedGroupsTable to handle visibility toggling for WhatsApp groups.
- Implemented setWhatsappGroupVisibility function to manage group visibility state.
- Added UI elements for displaying visibility status and handling user interactions.
- Updated routing to include a dedicated page for managing tags.
- Added PublishTagGroupsDialog to manage the visibility and untagging of groups with a specific tag. - Created TagGroupsPage to display groups associated with a flat tag, allowing for search and management. - Introduced useLabelGroupRows hook to streamline the retrieval of group rows based on labels. - Enhanced CreateEditGroupDialog to include an option to hide groups until published. - Updated CombinedGroupsTable to handle visibility toggling for WhatsApp groups. - Implemented setWhatsappGroupVisibility function to manage group visibility state. - Added UI elements for displaying visibility status and handling user interactions. - Updated routing to include a dedicated page for managing tags.
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
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. Comment |
BIA3IA
added
the
status: blocked
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The new tag route’s category redirect can generate invalid/unencoded URLs for labels containing spaces or other characters, which can break navigation.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR introduces a “publication” workflow built around reserved release- labels, adding a dedicated tag page to manage/publish those batches and extending group visibility controls (including WhatsApp) to support “hide until published” and later publishing.
Changes:
- Added
/dashboard/web/tags/$tagroute +TagGroupsPageto browse/manage groups for a flat tag (including release/publication tags). - Implemented publishing flow for release tags (unhide + untag in bulk) and refined label creation/rename validation to reserve
release-. - Added shared optimistic visibility-toggle hook and extended WhatsApp UI/server functions to support hide/visibility toggling and “hide until published” on creation.
File summaries
| File | Description |
|---|---|
| src/routeTree.gen.ts | Adds generated route entries for the new /dashboard/web/tags/$tag page. |
| src/routes/dashboard/web/tags/$tag.tsx | New tag route with redirect for categories and loader for tag page data. |
| src/hooks/use-group-visibility-toggle.ts | New shared optimistic visibility toggle hook (mutation + rollback + refresh). |
| src/features/whatsapp/whatsapp-groups-page.tsx | Adds WhatsApp visibility toggle UI and surfaces mutation/refresh errors. |
| src/features/whatsapp/whatsapp-group-fields.tsx | Adds optional “Hide until published” checkbox field for create flows. |
| src/features/whatsapp/groups.functions.ts | Adds hide support on create/edit and introduces setWhatsappGroupVisibility server fn. |
| src/features/whatsapp/create-edit-group-dialog.tsx | Adds “hide until published” option when creating a WhatsApp group. |
| src/features/telegram/groups-table.tsx | Refactors Telegram visibility toggle to use shared hook. |
| src/features/groups-by-label/use-label-group-rows.ts | New shared hook to derive combined rows for exact-label pages (category/tag). |
| src/features/groups-by-label/tag-groups-page.tsx | New flat-tag page UI (search + publish dialog + combined table). |
| src/features/groups-by-label/publish-tag-groups-dialog.tsx | New “publish release label” dialog (unhide + untag batch). |
| src/features/groups-by-label/groups-by-label-page.tsx | Refactors row derivation to useLabelGroupRows; passes allLabels onward. |
| src/features/groups-by-label/combined-groups-table.tsx | Adds WhatsApp visibility toggle and refactors Telegram toggle to shared hook. |
| src/features/groups-by-label/add-group-to-label-dialog.tsx | Adds multi-tag selection + optional creation flow gating + “hide on create”. |
| src/features/group-labels/rename-label-dialog.tsx | Prevents renaming publication labels / into the reserved release- prefix. |
| src/features/group-labels/label-tree.ts | Introduces RELEASE_LABEL_PREFIX, hasReleaseLabelPrefix, and isReleaseLabel. |
| src/features/group-labels/label-tree-selector.tsx | Splits “Attributes” vs “Publications” and supports tagsOnly mode. |
| src/features/group-labels/group-labels.validation.ts | Reserves release- from normal labels; adds createReleaseLabelInput; blocks renaming publications. |
| src/features/group-labels/group-labels.functions.ts | Adds createReleaseLabel server function. |
| src/features/group-labels/group-labels-page.tsx | Updates labels management UI to separate Attributes vs Publications; links to tag pages. |
| src/features/group-labels/group-label-card.tsx | Adds optional link target for label badge. |
| src/features/group-labels/add-tag-dialog.tsx | Adds publication creation mode (creates release labels from a name). |
| src/components/ui/checkbox.tsx | New shared Checkbox UI component used by “hide until published”. |
Review details
- Files reviewed: 23/23 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
BIA3IA
removed
the
status: blocked
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
lorenzocorallo
approved these changes
Sep 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.