Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unable to add default webhook for Telegram #7932

Closed
2 of 7 tasks
rchavik opened this issue Aug 22, 2019 · 1 comment · Fixed by #7972
Closed
2 of 7 tasks

Unable to add default webhook for Telegram #7932

rchavik opened this issue Aug 22, 2019 · 1 comment · Fixed by #7972
Labels
type/bug
Milestone
1.9.3

Comments

@rchavik
Copy link

rchavik commented Aug 22, 2019 •
edited
Loading

  • Gitea version (or commit ref): 1.9.1
  • Git version: 2.7.4
  • Operating system: Linux 4.4.0-116-generic 140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

Tried adding Telegram as a default webhook resulted in a empty tab pane:

Site Administration -> Default Webhooks -> Add Webhook -> Telegram

Note: Adding Telegram webhook manually by repository does work.

Screenshots

image
@lunny lunny added the type/bug label Aug 22, 2019
@gary-kim
Copy link
Member

Looks like that was forgotten when the webhook was added. I'll make a PR with a fix.

gary-kim added a commit to gary-kim/gitea that referenced this issue Aug 25, 2019
@gary-kim
Fix adding default Telegram webhook
9da4910 
Fixes go-gitea#7932

Signed-off-by: Gary Kim <gary@garykim.dev>
@gary-kim gary-kim mentioned this issue Aug 25, 2019
Merged
@lunny lunny added this to the 1.9.3 milestone Aug 25, 2019
techknowlogick pushed a commit that referenced this issue Aug 26, 2019
@gary-kim @techknowlogick
Fix adding default Telegram webhook (#7972)
541fab1 
* Fix adding default Telegram webhook

Fixes #7932

Signed-off-by: Gary Kim <gary@garykim.dev>

* Run goimports

Signed-off-by: Gary Kim <gary@garykim.dev>
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
type/bug
Projects
None yet
Milestone
1.9.3
Development

Successfully merging a pull request may close this issue.

Fix adding default Telegram webhook gary-kim/gitea
3 participants
@rchavik @lunny @gary-kim