Thanks your PR! It's quite a lot changes, look good so far. But I would recommend to modify .less file rather than gogs.css.

Great thanks! Ah yes... I was thinking those css files were generated, but for some reason I never stumbled onto the less files in my searching ... doh!

I've rebased now and made those changes.

This PR involves a lot changes, I have to take a little time to go through.

Sure! Yeah even I didn't think as many changes would be needed when I started, but hopefully adding additional hooks should be more flexible now.

Glad to make any additional changes. Still very new to go myself.

I believe your code is working, but just for reviewing some design pattern.

Other than that looks good, please resolve the conflicts then re-PR. I'll merge it as one of 0.5 release feature!

Okay all set! Thanks for reviewing.

Also as a heads-up - I have one more PR which compares two commits that I will submit in once this is merged.

Thanks!

I'm getting this error in template: template: repo/settings/slack_hook:1:91: executing "repo/settings/slack_hook" at <eq .HookType "slack">: error calling eq: invalid type for comparison

I don't know why, but suddenly worked.

Looks prefect!

I've pushed a mirror fix for file name from xx_hook.tmpl to hook_xx.tmpl.

I have noticed that the Slack webhook fails with the following message:

Delivery: Post %20https://hooks.slack.com/services/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%20: unsupported protocol scheme ""

I did a curl with the payload and it's all good. I think it might be related to the %20 at the start.

Edit: This is totally on me. I, of course, had a space at the start. I am leaving this up just in case anyone is as silly as me.

@drwlrsn I think it is because you have a space in the input field?

does the current slack support push issues to slack? it only seems to deal with commits. TIA

@dansteingart , no it does not..

@unknwon OK thanks. I'll try to hack the planet and see if I can get it to happen

Sorry for digging this up again, but would it be possible to make the messages pushed to Slack localizable - or even better, customizable? Currently, they are hard coded in english. It would be nice to have translations here or the possibility to set the strings using variables in the Gogs admin area.
I'll open a separate issue if that's more suited.

@Radiergummi good idea... please file another issue for this suggestion!