| Summary: | Undeployment failure notification duplicated | ||
|---|---|---|---|
| Product: | [Retired] JBoss BPMS Platform 6 | Reporter: | Zuzana Krejčová <zkrejcov> |
| Component: | Business Central | Assignee: | Maciej Swiderski <mswiders> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Zuzana Krejčová <zkrejcov> |
| Severity: | low | Docs Contact: | |
| Priority: | low | ||
| Version: | unspecified | CC: | kverlaen, lpetrovi, mswiders |
| Target Milestone: | ER5 | ||
| Target Release: | 6.0.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-08-06 20:12:27 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
|
Description
Zuzana Krejčová
2013-09-16 13:07:45 UTC
Zuzana, the idea of having this information sort of duplicated is because Problems panel could be: - closed (as user might close it) - not visible (very on the bottom of the page) and thus giving a short "fly over" notification might be very useful when looking for what happened. It's not meant to provide user with detailed information what went wrong but more of indicating that it failed. To be consistent with success notification. Removing this notification might lead to more issues when when having it as duplicate. wdyt? (In reply to Maciej Swiderski from comment #1) > Zuzana, the idea of having this information sort of duplicated is because > Problems panel could be: > - closed (as user might close it) > - not visible (very on the bottom of the page) > > and thus giving a short "fly over" notification might be very useful when > looking for what happened. It's not meant to provide user with detailed > information what went wrong but more of indicating that it failed. To be > consistent with success notification. Removing this notification might lead > to more issues when when having it as duplicate. > > wdyt? Well, that sounds reasonable. In this case, I think it would be even better if the "fly over" notification was really short - like "Undeployment failed." - and have some reason for this in the Problems panel. It is quite hard to catch anything more than that from the "fly over" notification anyway. As a user, it makes me a bit nervous that there is some kind of notification and I cannot read it all, because it disappears before I finish reading it. Would it be possible to bring up the Problems panel in the Deployments perspective in case of various failures? So that, if something went wrong, user would get the short notification and even if he closed the Problems panel earlier, he could then look for more information in there, as it would be brought up again (the short notification then serving as a kind of a pointer that "hey, something's wrong, look at the bottom"). I don't think that those "fly over" notifications are very good for detailed information, so keeping those messages as short as possible might still be best, if we can give the details some other way. This would probably apply to manual deployment of units as well btw. +1 for making the "fly over" notification to just indicate that state of the deployment (Success or Failure) and then the problems panel would contain details. I'll check if we can reopen the problems panel in case of an error although it might be bit annoying if someone closed it explicitly and we force it to reopen. Build & deploy and manual deployments rely on the same services so change in one place will apply to both. updated notification to contain only information about the failure and reference to problems panel. master https://github.com/droolsjbpm/jbpm-console-ng/commit/0813713c5f9b4a79ce7ac0142202723a9a4ee3e3 6.0.x https://github.com/droolsjbpm/jbpm-console-ng/commit/d98ffcc98990ef71b2442274a00122b062d3ca41 since user can decide to close the Problems panel we should not reopen it and thus marking it as modified for now. If there will be explicit need to reopen it we can revisit this case. New deployment errors are fixed, undeploy errors are not (the same should be done for them). Zuzana, could you please provide bit more information on what is missing and what undeployment errors you have seen? What I was able to check was that when you try to undeploy deployment that has active process instances then the error will be present in "Problems" panel. (In reply to Maciej Swiderski from comment #6) > What I was able to check was that > when you try to undeploy deployment that has active process instances then > the error will be present in "Problems" panel. That's exactly it. There's the error in the Problems panel and the user also gets pretty much the same, long message via the UberFire "fly over" notification. Your commit only changed the ....deployment.newunit.NewDeploymentPresenter. The same should be done for ....deployment.list.DeploymentUnitsListPresenter. There is still this (line 126 on current 6.0.x branch): view.displayNotification( "Error: Undeploy failed " + throwable.getMessage() ); whereas the fixed code for ....deployment.newunit.NewDeploymentPresenter has this for the "fly over" notification: view.displayNotification( "Error: Deploy failed, check Problems panel " ); Zuzana, good catch. fixed now. jbpm-console-ng master: https://github.com/droolsjbpm/jbpm-console-ng/commit/d7f9512f7dfaffddbf61ea9b2b58d5bcfa8232b7 6.0.x https://github.com/droolsjbpm/jbpm-console-ng/commit/383f86ff5a6cce3bcd28a0ce83b3c51af34a867b |