Not ready to merge - just want this open for code review as I begin working on it.

FirehoseCommunity / DEFCON

6 stars 4 forks source link

Not ready to merge - just want this open for code review as I begin working on it. #139

Open robertsapunarich opened 8 years ago

robertsapunarich commented 8 years ago

Reinstall github_api, omniauth, omniauth-github, and figaro gems.

kenmazaika commented 8 years ago

Looks like there's an additional file you checked-in called git. Only changes appear to be in the Gemfile.

robertsapunarich commented 8 years ago

I think I'm on the right path so far, but when I follow the link in the new user session view to sign in with github it takes me to an error 404 page on github. Any thoughts on why this might be? My hunch is that it's related to DEFCON being invite only. I have to step away from this for now but am going to plug away at this some more tomorrow. Also, if anyone has suggestions for testing API integration that would be greatly appreciated.

kenmazaika commented 8 years ago

It could be the invite-only nature, or it could just be that the gem isn't configured properly with the API keys. I integrated with GitHub oauth in the past, and it was a pretty difficult task. It's really not intuitive at all.