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.

Sign up for GitHub

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

Jump to bottom

Move data storage to Firebase #5

Closed
acorncom opened this issue Mar 23, 2016 · 2 comments
Closed

Move data storage to Firebase #5

acorncom opened this issue Mar 23, 2016 · 2 comments
Labels
Help Wanted Extra attention is needed
Milestone
MVP

Comments

@acorncom
Copy link
Member

acorncom commented Mar 23, 2016 •
edited
Loading

The current workingOn should end up pulling from /issues/<issueId>/appPayload whereas all other data will be pulled from /issues/<issueId>/githubPayload
@acorncom acorncom modified the milestone: Future Mar 25, 2016
@acorncom acorncom changed the title Consider moving data storage for this to something more permanent Move data storage to Firebase Apr 22, 2016
@acorncom acorncom modified the milestones: MVP, Future Apr 22, 2016
@acorncom
Copy link
Member Author

acorncom commented May 9, 2016

We'll also want all tests passing

@acorncom acorncom mentioned this issue May 9, 2016
Closed
9 tasks
@acorncom
Copy link
Member Author

acorncom commented May 9, 2016

As part of this, we'll need to produce the proper URLs to link to for each repo, as we're going to build that ourselves instead of using the html_url (and equivalent) data that Github sends in web hooks. Doing that keeps us safer from XSS attacks (and potential water-hole attacks against folks using this tool)

@acorncom acorncom added Help Wanted Extra attention is needed and removed Help Wanted Extra attention is needed labels May 9, 2016
@acorncom acorncom mentioned this issue May 12, 2016
Merged
acorncom pushed a commit that referenced this issue Sep 16, 2018
@EndangeredMassa
Merge pull request #5 from EndangeredMassa/smassa/remove-old-items
4c22f25 
remove "items"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Help Wanted Extra attention is needed
Projects
None yet
Milestone
MVP
Development

No branches or pull requests
1 participant
@acorncom