pretenders / ployst

The ployst core repo
1 stars 0 forks source link

Create web hook from admin. #76

Closed alexcouper closed 9 years ago

alexcouper commented 9 years ago

This contains ability to create a web hook from the admin page.

alexcouper commented 9 years ago

TODO

alexcouper commented 9 years ago

@txels This is ready for review.

alexcouper commented 9 years ago

Tested locally using pagekite

txels commented 9 years ago

While testing the code in this PR (which works nicely) I was debugging the hook and realised that it's not really working:

txels commented 9 years ago

I will push my fix to the first issue (fetch repo by URL). Maybe you can work on the second one @alexcouper ?

alexcouper commented 9 years ago

Re: second issue - just to be clear: we just want all branches to be stored in the DB for now, regardless of name etc. Is that right?

txels commented 9 years ago

I think so. So that the current UI is self-contained. The project provider settings (branch patterns) cannot be set via the UI, and features do not exist yet. So the only MVP is keep branch info up to date with what we get from the hook.

alexcouper commented 9 years ago

Second issue raised in #83