Skip to content
This repository has been archived by the owner on Jul 4, 2019. It is now read-only.

Initial setup #1

Closed
5 of 11 tasks
acorncom opened this issue May 2, 2016 · 1 comment
Closed
5 of 11 tasks

Initial setup #1

acorncom opened this issue May 2, 2016 · 1 comment

Comments

@acorncom
Copy link
Member

acorncom commented May 2, 2016

This issue is part of the initial MVP rollup issue for ember-learn/ember-help-wanted#7

I believe we'll want to have an active web hook listening for this payload:
https://developer.github.com/v3/activity/events/types/#issuesevent

Based on that information, we would need to confirm that we're actively tracking this repository and then update the appropriate issue tags in Firebase.

We'll want to handle the following cases:

  • closed
  • reopened
  • labelled (mostly done)
  • unlabeled
  • renaming an issue on Github updates the title on Firebase
  • ensure that creating an issue (with pre-existing label) triggers the labelled webhook (I would assume so, but am not positive)

I have setup a repo here: https://github.com/acorncom/ember-hitlist-tester to help with handling web hook notifications. If someone would like to be a contributor on it so they can add / remove labels, just let me know ;-)

Still needed:

  • add logic for filtering issues down by repo (and watched tags per repo)
  • refactor issue handler to have the pre-configured data store client injected (or passed in) from the index.js file (see PR WIP: Refactoring to models with constructors #3)
  • tests
  • Firebase config
  • work out Firebase data path. I was aiming for issues/repo-org/repo-name/<issue-id>, which should theoretically allow for doing a set for a given path (making updates completely controlled from our webhook) instead of needing to do updates. Would we then track "who is working on this issue" at a different path? Or should we move the issue down a level to /issues/repo-org/repo-name/<issue-id>/payload with /issues/repo-org/repo-name/<issue-id>/workingOn tracking contributors? How does this fit with setting Firebase permissions?
@acorncom
Copy link
Member Author

acorncom commented May 9, 2016

We're ready here ...

@acorncom acorncom closed this as completed May 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant