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

Welcome Page: Log In #7

Open
2 tasks
dionew1 opened this issue Jul 21, 2020 · 0 comments · Fixed by #29
Open
2 tasks

Welcome Page: Log In #7

dionew1 opened this issue Jul 21, 2020 · 0 comments · Fixed by #29
Labels
Core Core project requirement paired commit This issue should be completed as a pair. solo spike first Should timebox to research individually before discussing with partner.

Comments

@dionew1
Copy link
Contributor

dionew1 commented Jul 21, 2020

As a user,
When I visit the root path,
I should see a button to Log In with Google

Details: Implement Google OAuth 2.0 For the application, you will need the user to authorize access to their email or profile, as well as, calendar events. As part of the OAuth process, you will need to include the appropriate scopes for the users authorization. This documentation may be helpful.

Tips: Testing this is tricky. Don't worry about testing during the spike. In fact, TDD'ing this is really tricky if you've never done it before. Get a working implementation first even when you are pairing and backfill a test. You should not try to fill out the form and click buttons on Google in your tests. Omniauth gives us a special way to test this. Use these docs as a starting point:

  • Uses OmniAuth Google DO NOT hand roll the OAuth handshake
  • Tests the handshake using OmniAuth mock_auth (see above tips section).
@dionew1 dionew1 added OAuth paired commit This issue should be completed as a pair. solo spike first Should timebox to research individually before discussing with partner. labels Jul 21, 2020
@dionew1 dionew1 changed the title Log In Welcome Page: Log In Jul 21, 2020
@dionew1 dionew1 added the Core Core project requirement label Jul 21, 2020
@dionew1 dionew1 reopened this Sep 10, 2020
@dionew1 dionew1 removed the OAuth label Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Core project requirement paired commit This issue should be completed as a pair. solo spike first Should timebox to research individually before discussing with partner.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant