Bug 33327 - Set UCR variables for start site when installing a new app
Set UCR variables for start site when installing a new app
Status: CLOSED FIXED
Product: UCS extended documentation
Classification: Unclassified
Component: Developer documentation
unspecified
Other Linux
: P5 normal (vote)
: UCS 3.2
Assigned To: Dirk Wiesenthal
Moritz Muehlenhoff
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-12 14:23 CET by Stefan Gohmann
Modified: 2015-04-01 13:28 CEST (History)
4 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):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2013-11-12 14:23:01 CET
This should be described in the Dev Guide in the App Center chapter.

+++ This bug was initially created as a clone of Bug #32609 +++

The App Center needs to set corresponding UCR variables in order to make the app appear in the UCS start site.

We also need a little python script that allows to update all UCR variables in order to reflect the current status of installed apps. This script also needs to be executed once upon update (otherwise no link will be visible on the start site).
Comment 1 Stefan Gohmann univentionstaff 2013-11-12 14:23:29 CET
Important for the app upgrades.
Comment 2 Dirk Wiesenthal univentionstaff 2013-11-13 14:23:58 CET
Already added a while ago. See app:integration-by-app-center and app:iniFile
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2013-11-14 11:34:34 CET
The chapter "Extending the overview page" should also be updated. This is currently confusing as it describes the old way of extending the overview site.
Comment 4 Moritz Muehlenhoff univentionstaff 2013-11-14 11:41:40 CET
(In reply to Sönke Schwardt-Krummrich from comment #3)
> The chapter "Extending the overview page" should also be updated. This is
> currently confusing as it describes the old way of extending the overview
> site.

There's already Bug 33328 for that
Comment 5 Dirk Wiesenthal univentionstaff 2013-11-14 12:40:41 CET
(In reply to Moritz Muehlenhoff from comment #4)
> (In reply to Sönke Schwardt-Krummrich from comment #3)
> > The chapter "Extending the overview page" should also be updated. This is
> > currently confusing as it describes the old way of extending the overview
> > site.
> 
> There's already Bug 33328 for that

Thanks.
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2013-11-15 17:13:51 CET
The current developer guide states, that the UCR variable has to be named
  ucs/web/overview/service/<appid>/*
but the code looks like the correct name would be
  ucs/web/overview/entries/service/<appid>/*

What about entries for the admin part?
  ucs/web/overview/entries/admin/<appid>/*
Comment 7 Sönke Schwardt-Krummrich univentionstaff 2013-11-15 17:32:17 CET
A fully equiped example would also be nice.
Comment 8 Dirk Wiesenthal univentionstaff 2013-11-19 11:46:43 CET
(In reply to Sönke Schwardt-Krummrich from comment #6)
> The current developer guide states, that the UCR variable has to be named
>   ucs/web/overview/service/<appid>/*
> but the code looks like the correct name would be
>   ucs/web/overview/entries/service/<appid>/*

Yes, fixed.

> What about entries for the admin part?
>   ucs/web/overview/entries/admin/<appid>/*

Briefly described in ini-file part (UCSOverviewCategory=admin), I do not think that this is important for app vendors. But if, see below:

(In reply to Sönke Schwardt-Krummrich from comment #7)
> A fully equiped example would also be nice.

Yes. This is "only" the App Center part. The fully fledged documentation should be resolved by Bug#33471. Maybe one can even remove part of the App Center documentation when this has been done.
Comment 9 Moritz Muehlenhoff univentionstaff 2013-11-20 10:23:30 CET
Ok