Skip to content

feat: add loading spinner, error messages, and retry on async operations - #36

Open
anuza08 wants to merge 1 commit into
KDeekshita:mainfrom
anuza08:feat/async-feedback-ui
Open

feat: add loading spinner, error messages, and retry on async operations#36
anuza08 wants to merge 1 commit into
KDeekshita:mainfrom
anuza08:feat/async-feedback-ui

Conversation

@anuza08

@anuza08 anuza08 commented May 17, 2026

Copy link
Copy Markdown
  • Add loading spinner during async operations
  • Show error banner with descriptive message on failure
  • Add Retry button to re-attempt the last failed action

Closes #32

@KDeekshita

Copy link
Copy Markdown
Owner

Thanks for the contribution! This is a good direction, but please make a few cleanup changes before merge:

  • Remove duplicate or unrelated changes
  • Make sure the PR only solves the linked issue
  • Run formatting/checks locally
  • Update the branch with latest main if needed

After these fixes, I’ll re-review.
@anuza08

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@anuza08
anuza08 force-pushed the feat/async-feedback-ui branch from f5010e3 to 11815ee Compare May 19, 2026 09:11
@anuza08

anuza08 commented May 19, 2026

Copy link
Copy Markdown
Author

@KDeekshita Please review the PR, and let me know if any changes are required

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