Financial-Times / tako

🐙 A GitHub App that provides an API listing the repositories it is installed on.
MIT License
4 stars 0 forks source link

Refresh the Repository Store on webhook events and startup for all installation instances. #84

Closed adambraimbridge closed 5 years ago

adambraimbridge commented 5 years ago

Previously, the application listened to GitHub webhook events and added/removed individual items from a repository_store. However, when filtering by topic it made a call to the GitHub API. This is because the GitHub webhook event data does not include topic values for repositories.

This update makes the application completely refresh the repository_store for all webhook events and startup. This loads the appropriate topic data. Filtering is now based on the repository store, with no requirement to call the GitHub API.

adambraimbridge commented 5 years ago

image

sjparkinson commented 5 years ago

This has my blessing, but only if it gets a description and a gif.

adambraimbridge commented 5 years ago

Thanks Simon ❤️

... yeah, I'll sort out the webhook at https://github.com/organizations/financial-times/settings/apps/tako/permissions