Bug 54460 - App Center: Reason why certain servers are excluded is not shown
App Center: Reason why certain servers are excluded is not shown
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-1-errata
Assigned To: Johannes Lohmer
Florian Best
https://git.knut.univention.de/univen...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-02-16 10:47 CET by Dirk Wiesenthal
Modified: 2022-06-15 16:19 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.029
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments
This is how it looks right now (40.97 KB, image/png)
2022-03-03 11:54 CET, Dirk Wiesenthal
Details
... and this is how it should look like (48.41 KB, image/png)
2022-03-03 11:54 CET, Dirk Wiesenthal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2022-02-16 10:47:07 CET
The reason is collected, but then not shown. The names in the code don't match:

page.widgets.push({
  type: Text,
  name: `chooseHosts_removeExplanation_${app.id}`,
  content: removeExplanation,
  'class': 'umcAppChooseHostWizard__removeExplanation'
});
					page.layout.push(`removeExplanation_${app.id}`);

Normally, the ChooseHostWizard should show something like:
%s was removed from the list because the App is installed on this host.
Comment 1 Dirk Wiesenthal univentionstaff 2022-03-03 11:54:20 CET
Created attachment 10924 [details]
This is how it looks right now
Comment 2 Dirk Wiesenthal univentionstaff 2022-03-03 11:54:42 CET
Created attachment 10925 [details]
... and this is how it should look like
Comment 3 Johannes Lohmer univentionstaff 2022-04-27 13:20:37 CEST
univention-appcenter (9.0.2-76)
2381be6fce2a | Bug #54460: show why servers may be excluded from app-installation menu

-       page.layout.push(`removeExplanation_${app.id}`);
+       page.layout.push(`chooseHosts_removeExplanation_${app.id}`);
Comment 4 Florian Best univentionstaff 2022-05-31 22:03:29 CEST
OK: reason is shown again, looks good with the new UCS 5.0 design
OK: Code-Review
OK: YAML