Skip to content

Improved UI toasting of API errors#293

Merged
dbutenhof merged 11 commits intocloud-bulldozer:mainfrom
khandrew-redhat:toast
Feb 27, 2026
Merged

Improved UI toasting of API errors#293
dbutenhof merged 11 commits intocloud-bulldozer:mainfrom
khandrew-redhat:toast

Conversation

@khandrew-redhat
Copy link
Copy Markdown
Collaborator

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

Toast messages now do the following:

  • Account for different json structures when extracting message
  • Do not allow the pages to hand when error occurs
  • Fade in and out, lasting no longer than five seconds when displayed
  • Displays Axios error message instead of basic HTTP response

Related Tickets & Documents

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

Tests are located within axiosInstance.test.js

“khandrew-redhat” added 6 commits February 24, 2026 17:21
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Copy link
Copy Markdown
Collaborator

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

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

One real ESlint bug ... generally, this looks pretty cool, although I think a few bits could be cleaned up.

Did you hand-code all this (especially the neat CSS), or did you leverage an AI assistant? If you used AI, we're supposed to be tagging files with an "<assisted|created> by <cursor|claude|gemini|etc>" comment at the head of the file.

@dbutenhof dbutenhof changed the title Toast Improved UI toasting of API errors Feb 27, 2026
“khandrew-redhat” added 2 commits February 27, 2026 09:20
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
“khandrew-redhat” added 2 commits February 27, 2026 12:03
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
Signed-off-by: “khandrew-redhat” <“khandrew@redhat.com”>
@khandrew-redhat
Copy link
Copy Markdown
Collaborator Author

Removed error-handling e2e test since the webpage it was for is removed as well.

@dbutenhof dbutenhof merged commit 623cdf7 into cloud-bulldozer:main Feb 27, 2026
11 checks passed
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