Bug 49208 – Fix the naming for entry_uuid and id in IDM push chapter

Bug 49208 - Fix the naming for entry_uuid and id in IDM push chapter


Summary:	Fix the naming for entry_uuid and id in IDM push chapter

Status:	CLOSED FIXED

Product:	UCS extended documentation
Classification:	Unclassified
Component:	App Provider
Version:	unspecified
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	Nico Gulden
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2019-04-01 11:17 CEST by Nico Gulden
Modified:	2019-04-29 22:07 CEST (History)
CC List:	2 users (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?:	2: Will only affect a few installed domains
How will those affected feel about the bug?:	5: Blocking further progress on the daily work
User Pain:	0.057
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:	Yes
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Nico Gulden

2019-04-01 11:17:06 CEST

See https://chat.univention.de/channel/appcenter?msg=9CFM3sfhECEfjekGD

The JSON example in https://docs.software-univention.de/app-provider.html#provision:push:mechanism is not correct. The text speaks about `entry_uuid`. The example tells about `id`. That should be aligned.

Comment 1 Nico Gulden

2019-04-24 14:37:32 CEST

According to the working listener_trigger script in OpenProject (https://appcenter.software-univention.de/univention-repository/4.3/maintained/component/openproject_20190315104812/listener_trigger) it looks like that the example is correct with the `id`.

Furthermore, the `entry_uuid` term in the text should be replaced with `id`.

Comment 2 Nico Gulden

2019-04-26 15:49:24 CEST

Updated the text and replaced `entry_uuid` with id. Added a hint that the `id` contains the value of the `entry_uuid` LDAP attribute.

The reason for this behavior is in https://git.knut.univention.de/univention/ucs/blob/2140b08ea7f0cc9906a25191f5a1321323a82ffe/management/univention-appcenter/scripts/univention-appcenter-listener-converter#L130

See https://git.knut.univention.de/univention/ucs/compare/4.4-0...ngulden%2F49208-app-doku-fix-naming-entry-uuid-example for comparison of the change.

Comment 3 Dirk Wiesenthal

2019-04-26 16:25:27 CEST

Yes, thanks.

Please merge

Comment 4 Nico Gulden

2019-04-26 20:45:17 CEST

Merged.

Comment 5 Nico Gulden

2019-04-26 20:45:27 CEST

Verified.

Comment 6 Philipp Hahn

2019-04-29 22:07:49 CEST

[master] eb7cbcd Bug #49218,Bug #45681,Bug #41669,Bug #47982,Bug #48152,Bug #48312,Bug #48527,Bug #48921,Bug #48973,Bug #48974,Bug #49057,Bug #49208,Bug #49026,Bug #46793: PUBLISH
 24 files changed, 108 insertions(+), 56 deletions(-)