Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modal stays when integration no longer exists #5155

Closed
mmuzikar opened this issue Apr 8, 2019 · 5 comments
Closed

Modal stays when integration no longer exists #5155

mmuzikar opened this issue Apr 8, 2019 · 5 comments
Assignees
Labels
cat/bug A bug which needs fixing closed/migrated group/ui User interface SPA, talking to the REST backend prio/p2 The priority of a bug. p1 means low source/qe Raised by QE

Comments

@mmuzikar
Copy link
Contributor

mmuzikar commented Apr 8, 2019

See also https://issues.jboss.org/browse/ENTESB-11431

This is a...


[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  
[ ] Documentation issue or request

Description

This happens due to the delay between user calling delete on integration and the integration no longer existing in the UI. The delay is sometimes too long and it seems that nothing is happening so the user decides to press delete again; the modal shows up and in that time the integration can be already deleted.

The part of this behavior I'd call bug is that the modal stays up and the cancel and the X button don't close the window, so you have to refresh the page.
When you click on any of those button you get these errors in the console
image

@mmuzikar mmuzikar added cat/bug A bug which needs fixing prio/p2 The priority of a bug. p1 means low source/qe Raised by QE labels Apr 8, 2019
@pure-bot pure-bot bot added the notif/triage The issue needs triage. Applied automatically to all new issues. label Apr 8, 2019
@heiko-braun heiko-braun added group/ui User interface SPA, talking to the REST backend and removed notif/triage The issue needs triage. Applied automatically to all new issues. labels May 28, 2019
@heiko-braun heiko-braun added this to the Sprint 47 (3/4) milestone May 28, 2019
@heiko-braun
Copy link
Collaborator

@riccardo-forina Another one we have to verify on react. Please close this issue when it's no longer relevant with the new user interface.

@gashcrumb
Copy link
Contributor

I tested this on the react side against staging. What I found is both times I deleted an integration the backend would crash. In the react codebase this causes the error handler page to be rendered, which effectively removes the dialog that I left open from the page.

So kinda inconclusive, I think it's gonna work better but I can't say for sure.

@riccardo-forina
Copy link
Collaborator

It works as it should on the React UI.

@mmuzikar
Copy link
Contributor Author

mmuzikar commented Aug 8, 2019

Reopening for verification

@mmuzikar mmuzikar reopened this Aug 8, 2019
@heiko-braun
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/bug A bug which needs fixing closed/migrated group/ui User interface SPA, talking to the REST backend prio/p2 The priority of a bug. p1 means low source/qe Raised by QE
Projects
None yet
Development

No branches or pull requests

4 participants