DRILL-2653: Improve web UI experience when there is an error in a sto… #343
Conversation
…rage plugin configuration
|
@sudheeshkatkam @jacques-n Can you please review? |
|
@sudheeshkatkam Just saw that you actually had a patch for this, I was looking at DRILL-2663 originally, and found the duplicate DRILL-2653 when I went to write the commit message. I think I'll merge this version because it is a little clearer for users in the case of repeated failures, but I didn't mean to ignore your patch. |
There was a problem hiding this comment.
This alert will show even if the update succeeds, right?
There was a problem hiding this comment.
Originally I didn't have the "Please retry:" I had added that after finding your patch. Previously, it would just show either the error or the success message and only in the case of the success message reload. Will push a fix
|
That's fine, Jason :) |
|
@sudheeshkatkam Message fixed, also added more colors :) |
|
+1 |
2 participants
…rage plugin configuration