note: refacotr models/webhook.go

[6543]

note for me:

refactor models/webhook.go to combine GetWebhooksByRepoID and GetWebhooksByOrgID to `GetWebhooks(opts SearchWebhooksOptions) .,...

// SearchWebhooksOptions holds the search options
type SearchWebhooksOptions struct {
...

[6543]

@spawn2kill ping

[6543]

and we can also include GetActiveWebhooksByOrgID

[hilariocoelho]

I'll take this. If you find any more cases like this add to this issue please.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

[6543]

@spawn2kill are you still interested in it?

[alexanderadam]

I'm not sure whether this could be related but it would be great if it would be made with focus on #1089, if someone decides to refactor models/webhook.go.

[6543]

@alexanderadam this is out of scope ...

[alexanderadam]

Okay, no problem 👍 I just wanted to mention it, in case it would be suitable.

[6543]

Did have a closer look into codebase: combining would have only minimal dedublication and would add complxity so this wont happen for now