Skip to content

fix(ui): add missing delete buttons for agents and teams#63

Merged
viettranx merged 1 commit into
nextlevelbuilder:mainfrom
Luvu182:fix/delete-buttons-agents-teams
Mar 6, 2026
Merged

fix(ui): add missing delete buttons for agents and teams#63
viettranx merged 1 commit into
nextlevelbuilder:mainfrom
Luvu182:fix/delete-buttons-agents-teams

Conversation

@Luvu182
Copy link
Copy Markdown
Contributor

@Luvu182 Luvu182 commented Mar 5, 2026

Summary

  • Add Delete button (trash icon + label) to agent and team cards on list pages
  • Wire onDelete callbacks to the existing ConfirmDialog state that was already present but never triggered
  • Add Delete button + confirmation dialog to agent and team detail page headers
  • After deletion, navigates back to the list page

Context

The list pages (AgentsPage, TeamsPage) already had deleteTarget state and ConfirmDialog components wired up, but no UI element ever called setDeleteTarget(). Detail pages had no delete option at all.

Backend delete endpoints (DELETE /v1/agents/{id} and teams.delete WS method) were already implemented and working.

Closes #17

Test plan

  • Open Agents list → verify Delete button appears on each agent card
  • Click Delete on a card → confirm dialog appears → confirm → agent is deleted
  • Open Agent detail → verify trash icon in header → click → confirm → deletes and navigates back
  • Repeat same tests for Teams
  • Verify delete buttons don't appear when onDelete is not passed (no regressions)

The list pages had ConfirmDialog for deletion wired up but no UI
element to trigger it. Detail pages had no delete option at all.

- Add delete button with trash icon to agent and team cards
- Wire onDelete callbacks to existing setDeleteTarget state
- Add delete button + confirm dialog to agent detail page header
- Add delete button + confirm dialog to team detail page header

Closes nextlevelbuilder#17
@viettranx viettranx merged commit 1ebbe53 into nextlevelbuilder:main Mar 6, 2026
1 of 2 checks passed
mrgoonie added a commit that referenced this pull request May 28, 2026
Implements 3 coalescing layers to handle rapid multi-attachment inbounds:
- Bus debouncer: delays inbound messages 1s, merges duplicates
- Web chat debouncer: buffers client-side inbound frames for batch RPC
- Telegram album aggregator: collects album members via AfterFunc+Stop timer

Drops media-bypass shortcut (forces 1s media floor). Aggregator enforces:
- AfterFunc+Stop timer discipline with ordered drain on stop
- 2-tuple (album_id, sender) keying for isolation
- Dual DoS caps: max 10 albums per sender, max 100 messages per album
- merged_message_ids dedup seeding across all 3 surfaces

Closes #63
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.

Title: [Bug/Feature Request] Missing "Delete" Action Buttons for Agents & Teams in Web UI

2 participants