Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

Flask-Login for user management #278

Merged
merged 6 commits into from
Jul 6, 2020

Conversation

alysivji
Copy link
Member

@alysivji alysivji commented Jul 6, 2020

  • use Flask Login to enable user sessions
  • set up login and logout views.
  • set up flask test client to use custom class provided by Flask-Login library

Resources

@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2020

Codecov Report

Merging #278 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #278      +/-   ##
==========================================
+ Coverage   97.26%   97.28%   +0.02%     
==========================================
  Files          68       68              
  Lines        1863     1880      +17     
==========================================
+ Hits         1812     1829      +17     
  Misses         51       51              
Impacted Files Coverage Δ
busy_beaver/app.py 93.87% <100.00%> (+0.12%) ⬆️
busy_beaver/apps/slack_integration/api/oauth.py 100.00% <100.00%> (ø)
busy_beaver/apps/slack_integration/models.py 100.00% <100.00%> (ø)
..._beaver/apps/slack_integration/oauth/oauth_flow.py 100.00% <100.00%> (ø)
...sy_beaver/apps/slack_integration/oauth/workflow.py 97.05% <100.00%> (-0.05%) ⬇️
busy_beaver/apps/web/views.py 100.00% <100.00%> (ø)
busy_beaver/extensions.py 100.00% <100.00%> (ø)

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 4b37742...09ce4fa. Read the comment docs.

@alysivji alysivji changed the title Flask-Login -- sessions cookies Flask-Login for user management Jul 6, 2020
@alysivji alysivji merged commit f18e0e7 into busy-beaver-dev:master Jul 6, 2020
@alysivji alysivji deleted the flask-login branch July 6, 2020 03:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants