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

🔐 OAuth #89

Merged
merged 3 commits into from
Jul 31, 2023
Merged

🔐 OAuth #89

merged 3 commits into from
Jul 31, 2023

Conversation

mrharpo
Copy link
Contributor

@mrharpo mrharpo commented Jul 26, 2023

OAuth

Initial OAuth implementation for user login

Closes #64

Screen Shot 2023-07-26 at 15 06 50

Usage

Add the following config values to .env.development

AUTH0_CLIENT_ID=
AUTH0_CLIENT_SECRET=
AUTH0_DOMAIN=
CHOWDA_SECRET=

@mrharpo mrharpo added enhancement ➕ New feature or request production 🎭 Relating to the production deployment labels Jul 26, 2023
@mrharpo mrharpo requested a review from afred July 26, 2023 22:12
@mrharpo mrharpo self-assigned this Jul 26, 2023
@mrharpo mrharpo temporarily deployed to Codecov July 26, 2023 22:13 — with GitHub Actions Inactive
@mrharpo mrharpo temporarily deployed to Codecov July 26, 2023 22:13 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2023 •
edited
Loading

Codecov Report

Merging #89 (41740c6) into main (f895303) will decrease coverage by 1.57%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   82.63%   81.06%   -1.57%     
==========================================
  Files          12       14       +2     
  Lines         380      433      +53     
==========================================
+ Hits          314      351      +37     
- Misses         66       82      +16
Files Changed Coverage Δ
chowda/auth/oauth.py 61.53% <61.53%> (ø)
chowda/views.py 69.56% <83.33%> (+0.33%) ⬆️
chowda/app.py 100.00% <100.00%> (ø)
chowda/auth/__init__.py 100.00% <100.00%> (ø)
chowda/config.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mrharpo mrharpo temporarily deployed to Codecov July 26, 2023 22:21 — with GitHub Actions Inactive
@mrharpo mrharpo temporarily deployed to Codecov July 26, 2023 22:21 — with GitHub Actions Inactive
@mrharpo mrharpo marked this pull request as ready for review July 31, 2023 16:36
afred
afred approved these changes Jul 31, 2023
View reviewed changes
Copy link
Contributor

@afred afred left a comment

Choose a reason for hiding this comment

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

👍

@afred afred merged commit eef7414 into main Jul 31, 2023
8 checks passed
@afred afred deleted the auth branch July 31, 2023 16:50
mrharpo added a commit that referenced this pull request Jul 31, 2023
@mrharpo
Merge branch 'main' into update-dependencies
77d249f 
* main:
  Displays sync history in dashboard (#86)
  🔐 OAuth (#89)
@mrharpo mrharpo added this to the v0.3 Pipeline 🚰 milestone Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@afred afred afred approved these changes

Assignees

@mrharpo mrharpo

Labels
enhancement ➕ New feature or request production 🎭 Relating to the production deployment
Projects
No open projects
Chowda 2023
Status: ✅ Done
Milestone
v0.3.0 Pipeline 🚰
Development

Successfully merging this pull request may close these issues.

User accounts
3 participants
@mrharpo @codecov-commenter @afred