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.

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

Track project webhooks and display on integration page #2755

Closed
11 tasks done
agjohnson opened this issue Mar 28, 2017 · 0 comments
Closed
11 tasks done

Track project webhooks and display on integration page #2755

agjohnson opened this issue Mar 28, 2017 · 0 comments
Assignees
Labels
Improvement Minor improvement to code

Comments

@agjohnson
Copy link
Contributor

agjohnson commented Mar 28, 2017

Webhook creation should add a webhook object to track the status of the project webhook, and allow operations on the webhooks. The integration APIs should give back an object id. We will track this and eventually allow for additional operations on the remote object or automatic updating.

  • Make a per oauth service update_webhook that uses the object id and URL to update the webhook object. It might make sense to break up some of the functionality around oauth services.
  • Figure out if we want a pattern that uses a through model for Integrations. This could either use a generic relation to actual integration types, or we could just make a generic skeleton for integration types.
  • UI for singular webhook resync
  • API webhook responses into an auto created webhook object
  • Difference between auto created webhooks from intermediate webhook views and explicit integration views. Don't show sync button?
  • Do something with the weird UI around missing a project webhook
  • Generic brute force webhook creation when missing new project bits?
  • Handle integration with missing data before sync operation. Perhaps syncing on a webhook like this defaults to trying to create the webhook? This should avoid recreating a webhook on our side though.
  • Filter some of the headers we don't want to display
  • Admin views

Later:

  • Create webhook UI? This would rely on the remote repo object connected to the project. We would use this to try to create a webhook
@agjohnson agjohnson added this to the Webhook admin dashboard milestone Mar 28, 2017
@agjohnson agjohnson self-assigned this Mar 28, 2017
@agjohnson agjohnson added Improvement Minor improvement to code Working labels Mar 28, 2017
agjohnson added a commit that referenced this issue May 10, 2017
Addresses #2708 by using a version of our extension that supports more than one
version of Sphinx.

Refs #1895
Refs #1850
Refs #2755
agjohnson added a commit that referenced this issue May 11, 2017
Addresses #2708 by using a version of our extension that supports more than one
version of Sphinx.

Refs #1895
Refs #1850
Refs #2755
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code
Projects
None yet
Development

No branches or pull requests

1 participant