Bug 46838 - [4.2] Missing log summary after dryrun
[4.2] Missing log summary after dryrun
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: HTTP-API (Kelvin)
unspecified
Other Mac OS X 10.1
: P5 normal (vote)
: UCS@school 4.2 v11
Assigned To: Daniel Tröder
Jürn Brodersen
:
Depends on: 46712
Blocks: 45019
  Show dependency treegraph
 
Reported: 2018-04-17 13:39 CEST by Daniel Tröder
Modified: 2018-09-11 11:33 CEST (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.023
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2018-04-17 13:39:12 CEST
+++ This bug was initially created as a clone of Bug #46712 +++

Currently a dryrun in the Web-Import do not show a summary like X created, Y modified, Z deleted and 0 errors.
Comment 1 Daniel Tröder univentionstaff 2018-07-06 11:27:49 CEST
Merged to 4.3 and built.

[4.2] 0b5c27cc7 Bug #46838: display summary after dry-run in UMC import module
[4.2] ba7b3350b Bug #46838: changelog
[4.2] 07b3d8513 Bug #46838: advisory

ucs-school-import (15.0.3-63)
Comment 2 Daniel Tröder univentionstaff 2018-07-30 15:26:39 CEST
Merged to 4.2, not 4.3 :)
Comment 3 Jürn Brodersen univentionstaff 2018-08-07 19:00:21 CEST
Same tests as in bug 46712 -> OK

YAML -> Fail

This bug isn't in the yaml. It looks like you removed that line with commit 5dac9f3ca7 ?

-> Reopen
Comment 4 Daniel Tröder univentionstaff 2018-08-08 10:12:16 CEST
Ah yes - probably happend in a merge.

[4.2 34b388950] Bug #46838: readd advisory
Comment 5 Jürn Brodersen univentionstaff 2018-08-08 11:58:23 CEST
(In reply to Daniel Tröder from comment #4)
> Ah yes - probably happend in a merge.
> 
> [4.2 34b388950] Bug #46838: readd advisory

Looks good now :)

-> Verified
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2018-09-11 11:33:37 CEST
UCS@school 4.2 v11 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.2v11-de.html

If this error occurs again, please clone this bug.