Bug 32163 - Show "prohibiting message" in popup dialog
Show "prohibiting message" in popup dialog
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 3.0
Other Linux
: P5 enhancement (vote)
: UCS 3.2
Assigned To: Dirk Wiesenthal
Alexander Kläser
: interim-2
Depends on: 31891
Blocks: 30077
  Show dependency treegraph
 
Reported: 2013-08-06 13:23 CEST by Stefan Gohmann
Modified: 2013-11-19 06:43 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
app general information page (116.27 KB, image/png)
2013-09-19 12:00 CEST, Erik Damrose
Details
app installation confirmation with bad button placement (66.00 KB, image/png)
2013-09-19 12:00 CEST, Erik Damrose
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2013-08-06 13:23:55 CEST
Currently the open button is not shown if the app is not installable for example if not enough RAM is available. The open button should be available and a popup dialog should show the reason why the app is not installable.
Comment 1 Stefan Gohmann univentionstaff 2013-08-15 10:21:45 CEST
As discussed with Alex, maybe the detail view of an app could be replaces with theses messages, so it would be more like a wizard.
Comment 2 Alexander Kläser univentionstaff 2013-08-27 16:29:27 CEST
Another solution would be as follows. Apps are rendered as separate page that overlays the app center page (→ instead of popup). Upon click on "install" button, a popup indicates that the app cannot be installed, e.g.:
* due to not enough RAM (→ the user can force the installation)
* due to a missing app (→ the user can confirm to install the other app, as well)
* due to a conflicting app (→ the user could have a link to open the other app in a new UMC tab?)
...
Comment 3 Dirk Wiesenthal univentionstaff 2013-09-05 13:17:12 CEST
Fixed in
  univention-management-console-module-appcenter 3.0.25-1.177.201309051116

Each test that fails may have a solution that is presented to the user (see Comment 2). Each test can force abortion or just emit a warning.
Comment 4 Dirk Wiesenthal univentionstaff 2013-09-05 14:26:08 CEST
This bug required huge changes in the App Center frontend and backend code. Some concepts did not work with the new code and I changed things out of scope of this bug. Thus I used Bug#31891.
Comment 5 Erik Damrose univentionstaff 2013-09-19 12:00:14 CEST
Created attachment 5444 [details]
app general information page

There are some issues with the current installation workflow:

Choosing to install an app presents the general information dialog with an 'Install' Button on the lower right. If i click that, installation checks are performed and on the next page, a list of 'packages-to-be-installed' is presented. On that page, the buttons to Install or Cancel are not below the <hline>, but above, with the list of packages. This can be very confusing when an app with many packages is to be installed, as one has to scroll down to see any controls. See attached screenshots.

I would suggest moving the Buttons below the <hline>, as on the general information dialog.
Comment 6 Erik Damrose univentionstaff 2013-09-19 12:00:49 CEST
Created attachment 5445 [details]
app installation confirmation with bad button placement
Comment 7 Erik Damrose univentionstaff 2013-09-20 10:20:19 CEST
Warnings about installation issues (e.g. 'Der Server hat versucht, sich mit den DC Backups zu verbinden. Folgende Hosts konnten nicht erreicht werden:') are also not directly visible, as they are located directly above the Install/Update Button.

Warnings should be moved to the top, above the package list.
Comment 8 Dirk Wiesenthal univentionstaff 2013-09-20 12:00:32 CEST
Buttons and warning moved in
  univention-management-console-module-appcenter 3.0.36-1.195.201309201157
Comment 9 Alexander Kläser univentionstaff 2013-09-27 14:29:35 CEST
I like the wizard like detail view now, it's really nice :) .
→ OK

A suggestion is shown in case the app should not be installed (e.g., due to lacking memory etc.), however, the app can be installed nevertheless.
→ OK

Changelog fine, maybe it would be nice to mention that the view of app details has been adapted.
→ OK
Comment 10 Stefan Gohmann univentionstaff 2013-11-19 06:43:47 CET
UCS 3.2 has been released:
 http://docs.univention.de/release-notes-3.2-en.html
 http://docs.univention.de/release-notes-3.2-de.html

If this error occurs again, please use "Clone This Bug".