Skip to content

feat(campaigns): remove Contacts tab from campaign detail page - #25

Merged
anujeet98 merged 2 commits into
mainfrom
feat/remove-campaign-contacts-tab
Aug 10, 2026
Merged

feat(campaigns): remove Contacts tab from campaign detail page#25
anujeet98 merged 2 commits into
mainfrom
feat/remove-campaign-contacts-tab

Conversation

@Rishavraaj

Copy link
Copy Markdown
Member

Summary

Removes the Contacts tab from the campaign detail page to simplify the view, as per the current sprint goal.

Changes

  • Removed contacts from Tab type, TAB_VALUES, and TABS array
  • Removed the Contacts panel conditional render
  • Contact-related query logic, columns, and components kept in place

Test plan

  • Open any campaign detail page and confirm only Sequence Editor and Analytics tabs are visible
  • Both remaining tabs work correctly
  • Navigating to ?tab=contacts via URL falls back to default Sequence Editor tab

@Rishavraaj
Rishavraaj force-pushed the feat/remove-campaign-contacts-tab branch from 428a8f9 to 40cf502 Compare August 10, 2026 04:54
Rishavraaj and others added 2 commits August 10, 2026 10:26
…ment

- Remove AddContactsDialog component and replace with inline contact management UI
- Update CampaignDetail to handle contact addition directly without modal dialog
- Simplify Apollo API integration by removing unused prospect search functionality
- Refactor query options to optimize contact-related data fetching
- Reduce modal complexity and improve UX by consolidating contact workflows

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Rishavraaj
Rishavraaj force-pushed the feat/remove-campaign-contacts-tab branch from 40cf502 to a112dbe Compare August 10, 2026 04:57
@anujeet98
anujeet98 merged commit 00b0f7c into main Aug 10, 2026
1 check passed
@anujeet98
anujeet98 deleted the feat/remove-campaign-contacts-tab branch August 10, 2026 06:15
@Rishavraaj Rishavraaj self-assigned this Aug 12, 2026
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