c81b26b0e5
refactor webhook *NewPost ( #20729 )
...
* refactor webhook *NewPost
* remove empty values
* always show errs.Message
* remove utils.IsValidSlackChannel
* move IsValidSlackChannel to services/webhook package
* binding: handle empty Message case
* make IsValidSlackChannel more strict
2 years ago
965861043a
Return the full rejection message and errors in flash errors ( #13221 )
...
Signed-off-by: Andrew Thornton <art27@cantab.net>
4 years ago
64133126cd
Update golangci-lint to version 1.31.0 ( #13102 )
...
This PR updates golangci-lint to the latest version 1.31.0.
The upgrade introduced a new check for which I've fixed or disabled most cases.
Signed-off-by: kolaente <k@knt.li>
4 years ago
74bd9691c6
Re-attempt to delete temporary upload if the file is locked by another process ( #12447 )
...
Replace all calls to os.Remove/os.RemoveAll by retrying util.Remove/util.RemoveAll and remove circular dependencies from util.
Fix #12339
Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: silverwind <me@silverwind.io>
4 years ago
089ccb0c80
Handle push rejection message in Merge & Web Editor ( #10373 )
...
* Handle push rejection message in Merge
* placate golangci-lint
* Fix sanitize, adjust message handling
* oops
* Oops
* Handle push-rejection in webeditor CRUD too
* Apply suggestions from code review
Co-authored-by: Lauris BH <lauris@nix.lv>
5 years ago
be48397945
Slack webhook channel name cannot be empty or just contain an hashtag ( #4786 )
6 years ago
311c83ad17
Allow adding collaborators with (fullname) ( #3103 )
...
* Allow adding collaborators with (fullname)
Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>
* Refactor username suffix to utils pkg
Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>
7 years ago
oliverpool
zeripath
kolaente
Lanre Adelowo
Sasha Varlamov