Bug 40782
| Summary: | Allow users to send system setup errors as feedback to Univention | ||
|---|---|---|---|
| Product: | UCS | Reporter: | Alexander Kläser <klaeser> |
| Component: | UMC - Setup wizard | Assignee: | Johannes Keiser <keiser> |
| Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
| Severity: | normal | ||
| Priority: | P5 | CC: | best |
| Version: | UCS 4.0 | ||
| Target Milestone: | UCS 4.1-2-errata | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| What kind of report is it?: | Feature Request | 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): | Error handling, Usability | |
| Max CVSS v3 score: | |||
| Bug Depends on: | |||
| Bug Blocks: | 40006 | ||
| Attachments: |
bug40782.patch
Screenshot |
||
|
Description
Alexander Kläser
*** Bug 37087 has been marked as a duplicate of this bug. *** Created attachment 7772 [details] bug40782.patch @Alex We should not offer to send feedback via email, right? (In a virtual machine one cannot send a email). → If yes, Johannes, please remove getSendErrorsAsMailButton() from the patch (the whole logic can be simplified then) 2. Should we display the errors inline or as a pop up? → As a pop up would have the advantage that we could use umc.tools._fooTraceback() - which also handles appearance. 3. If not 2. I think a field for a remark (editable) and a field with the traceback/errors (no editable) should be shown. The patch looks fine btw. Occurred errors can now be send via sysinfo/traceback and if possible also per mail r 70889 univention-system-setup (9.0.4-28) * Bug #40782: Allow users to send system setup errors as feedback YAML: r 70890 OK: Firefox OK: Chromium ~OK: The error box doesn't disappear after sending feedback. There is a small pop up which says 'Thanks for your help'. OK: from system-setup-browser OK: "Send as email button" works OK: "Send as email button" is only displayed when coming from external REOPEN: There is no field where the user can write some remark REOPEN: The entered email address is not set REOPEN: if above is fixed, it must be clear that the email address is send along with the request REOPEN: The feedback-email which we receive contains the traceback/error twice: * 1. in the traceback field * 2. in the remark field Created attachment 7807 [details]
Screenshot
(REOPEN): When sending the information to univention failed there is a nice pop up:
"""
Benachrichtigung
Das Versenden der Informationen zum Hersteller ist fehlgeschlagen. Sie können die Informationen auch per Mail schicken: Feedback als E-Mail senden
"""
The email button is also shown if we are in the system-setup-browser. Clicking on it causes firefox to pop up the window in the attached screenshot.
There is a scrollbar shown in the system-setup-browser if the error is longer than 10 lines. As discussed the errors are still displayed in-line but the feedback is handled via tools._handleTraceback. If the setup is local the traceback function from sysinfo/lib is used directly since emails can not be send. Also fixed the scrollbar showing when there are to many error lines. r 71084 univention-system-setup (9.0.4-32) YAML: r 71085 OK: HTML escaping OK: latest changes OK: scrollbar OK: YAML It would be nice if the email was prefilled but then we would need to touch the sysinfo module. |