Skip to content

Feature/inline approval ux squashed#1005

Closed
ANierbeck wants to merge 2 commits intoRightNow-AI:mainfrom
ANierbeck:feature/inline-approval-ux-squashed
Closed

Feature/inline approval ux squashed#1005
ANierbeck wants to merge 2 commits intoRightNow-AI:mainfrom
ANierbeck:feature/inline-approval-ux-squashed

Conversation

@ANierbeck
Copy link
Copy Markdown

Adds inline approval UI to both TUI and WebUI, resolving the UX issue where users had to navigate to a separate Approval menu that didn't show pending requests properly.

Changes

  • Added inline approval notifications directly in chat interfaces
  • Implemented keyboard shortcuts (A/R for approve/reject, ↑↓ for navigation)
  • Added risk level visualization with color-coded indicators
  • Integrated WebUI with approval API endpoints
  • Fixed undefined _audioFile property access error
  • Enhanced global state management for approvals
  • Added automatic polling for approval status updates

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (WebUI approval system fully functional)

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

This commit includes:
- Inline approval UI for both TUI and WebUI
- Keyboard shortcuts for quick approval/rejection (A/R keys)
- Risk level visualization and navigation
- WebUI API integration for approval actions
- Fixed undefined _audioFile property access
- Enhanced global state management for approvals
- Automatic approval loading and polling

Resolves the UX issue where users had to navigate to a separate
Approval menu that didn't show pending requests properly.
@ANierbeck
Copy link
Copy Markdown
Author

To many issues, I'll close this one and will try to provide a better one ...

@ANierbeck ANierbeck closed this Apr 7, 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.

1 participant