Skip to content

Chat attachment dropdown: add Deal option#13

Open
Baba-2001 wants to merge 2 commits intomainfrom
context-attachments
Open

Chat attachment dropdown: add Deal option#13
Baba-2001 wants to merge 2 commits intomainfrom
context-attachments

Conversation

@Baba-2001
Copy link
Contributor

@Baba-2001 Baba-2001 commented Mar 3, 2026

Summary

UI only. Chat attachment dropdown: add Deal option alongside Contact and Company. Users can attach deals as context when starting or continuing a chat; deal chips display with title and are hydrated on load.

E2E: attachment menu shows Contact, Company, and Deal; tests for selecting and adding one contact, company, and deal as context (chip visible).

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation / chore

Checklist

  • E2E tests for chat attachments (menu options + select contact/company/deal)
  • Lint/build passes
  • bd task linked (if applicable)

- Add deal type to ChatContextManager (entity type, fetch deals, deal items)
- Add Deal dialog title/description in ChatContextDialog
- Add 'Add deal' menu item in ChatContextMenuItems
- Hydrate deal context on chat page (first message and history load)

Made-with: Cursor
@Baba-2001 Baba-2001 changed the title Chat attachment: add Deal option + agent deal tools Chat attachment: add Deal option Mar 3, 2026
@Baba-2001 Baba-2001 changed the title Chat attachment: add Deal option Chat attachment dropdown: add Deal option Mar 3, 2026
- Attachment menu shows Contact, Company, and Deal options
- Can select and add one contact/company/deal as context; assert chip visible
- Scope attachment trigger to main to avoid opening sidebar user menu

Made-with: Cursor
: 'Add Companies'
: contextManager.dialogConfig.type === 'company'
? 'Add Companies'
: 'Add Deals'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of conditional statements, lets create a labelMap to render based on the type

: 'Select companies to add as context'
: contextManager.dialogConfig.type === 'company'
? 'Select companies to add as context'
: 'Select deals to add as context'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use LableMap instead

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