Skip to content

Conversation

@RubySpeeders
Copy link
Contributor

Description

DESCRIPTION_HERE

🔗 Reference

Relevant links (e.g. issue, design, etc.)

📋 Acceptance Criteria

Checked off by PR Assignees

  • user can sign in/sign out with google
  • button to sign in and out on the tasks pages (on left hamburger menu)

👩‍🔬 Test Instructions

Provided by PR Assignees

  1. Click "Get Started" to be taken to the tasks page
  2. Click the hamburger menu in the top left corner to open the drawer.
  3. Click "Sign In" to sign in with google
  4. Click the hamburger menu in the top left corner again to open the drawer.
  5. Click "Sign Out" to sign out.

🔎 Reviewer Checklist

Checked off by PR Reviewers

  • Merge destination is correct
  • Code is correct as understood and conforms to quality standards
  • Tests have been added where appropriate (unit, visual, end-to-end)
  • Acceptance Criteria have been met

Roles & Responsibilities

👨‍💻 Assignee

  • Initiator of this PR (be sure to set in GitHub UI)
  • Addresses feedback and change requests
  • Merges PR once approved (usually deletes branch unless develop or release)

👩‍💻 Reviewer

  • Invited to review PR by Assignee (via GitHub UI)
  • Is expected to complete a review and address followup

@RubySpeeders RubySpeeders self-assigned this Jun 8, 2021
@codecov
Copy link

codecov bot commented Jun 8, 2021

Codecov Report

Merging #11 (496e1c9) into develop (e7e9813) will decrease coverage by 18.56%.
The diff coverage is 65.55%.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop      #11       +/-   ##
============================================
- Coverage   100.00%   81.43%   -18.57%     
============================================
  Files           12       19        +7     
  Lines           79      167       +88     
  Branches        15       31       +16     
============================================
+ Hits            79      136       +57     
- Misses           0       31       +31     
Impacted Files Coverage Δ
src/redux/store.ts 100.00% <ø> (ø)
src/components/Login/Login.tsx 33.33% <33.33%> (ø)
src/hooks/useAuth.ts 38.09% <38.09%> (ø)
src/components/App/index.tsx 55.55% <42.85%> (-44.45%) ⬇️
src/components/Logout/Logout.tsx 71.42% <71.42%> (ø)
src/components/AppHeader/AppHeader.tsx 100.00% <100.00%> (ø)
...c/components/ButtonCreateTask/ButtonCreateTask.tsx 100.00% <100.00%> (ø)
src/components/Drawer/Drawer.tsx 100.00% <100.00%> (ø)
src/components/GoogleAuth/GoogleAuth.tsx 100.00% <100.00%> (ø)
src/redux/drawer/drawerSlice.ts 100.00% <100.00%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7e9813...496e1c9. Read the comment docs.

@RubySpeeders RubySpeeders requested a review from amtadros June 9, 2021 14:24
Copy link

@amtadros amtadros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Just left a few questions.

@amtadros amtadros requested review from blaketarter and roblafeve June 9, 2021 20:38
…dding styles to drawer to make absolute and therefore not move the main task page around
… to see if token exists, and if so, dispatch and sign in
@roblafeve
Copy link
Contributor

@RubySpeeders how can we be of help here? 😄

Comment on lines +10 to +12
client_id: Cypress.env("googleClientId"),
client_secret: Cypress.env("googleClientSecret"),
refresh_token: Cypress.env("googleRefreshToken"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RubySpeeders I think that we need to add these variables to the GitHub repo settings in order to successfully authenticate. This would be the information you have in your .env file I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants