diff options
author | Alfredo Sumaran <alfredo@gitlab.com> | 2016-12-10 01:49:08 +0300 |
---|---|---|
committer | Alfredo Sumaran <alfredo@gitlab.com> | 2016-12-10 01:49:08 +0300 |
commit | 3a66c2398ac3ae85b4703b2edd9f9adf62494759 (patch) | |
tree | ba5cafd02868ee23e9a6f1127a909639897d5567 /changelogs | |
parent | 4f365066330365391c11eeb408e884c021465a00 (diff) | |
parent | 3ce2ba1afb1dad707e1e3ac2bb5ff856d06f621e (diff) |
Merge branch 'issue_24020' into 'master'
fix display hook error message
## What does this MR do?
Fix hook error message display.
## Are there points in the code the reviewer needs to double check?
Probably Hook will return the plain multi line text. It is better `pre` tag than `h4`.
I thought of another good looking display, but I did not come up with it. :sweat:
## Why was this MR needed?
When the hook returns an STDERR, "Merge in progress" button spinning forever. Due to javascript's error.
## Screenshots (if relevant)
When update hook returns an error like
```
XXXXXXXX
YYYYYYYY
ZZZZZZZZ
```
![screen_shot](/uploads/8dac77972a309dfcb72c741dffbd7270/screen_shot.png)
## What are the relevant issue numbers?
Closes #24020
See merge request !7775
Diffstat (limited to 'changelogs')
-rw-r--r-- | changelogs/unreleased/issue_24020.yml | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/changelogs/unreleased/issue_24020.yml b/changelogs/unreleased/issue_24020.yml new file mode 100644 index 00000000000..87310b75296 --- /dev/null +++ b/changelogs/unreleased/issue_24020.yml @@ -0,0 +1,4 @@ +--- +title: "fix display hook error message" +merge_request: 7775 +author: basyura |