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

Feat/swagger auth #1141

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Feat/swagger auth #1141

wants to merge 7 commits into from

Conversation

Dawn0472
Copy link

add swagger authorization

  1. add auth button in swagger
  2. add auth url to get token (逾7天更新一次,若無token則新增,有token直接取得- one user vs one token)
  3. update core.models.Token(預設 one user vs multiple token)
  4. update BearerAuthentication function(swagger authorization)
  5. update api import BearerAuthentication

@josix josix self-requested a review May 31, 2023 04:19
@josix
Copy link
Collaborator

josix commented May 31, 2023

Hello @Dawn0472, thank you for working on this. There are some coding style issues, could you help me fix it, thanks!

@mattwang44 mattwang44 self-requested a review May 31, 2023 07:01
src/users/api/views.py Outdated Show resolved Hide resolved
@josix josix requested a review from erik1110 June 10, 2023 13:15
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 65.82% and project coverage change: +0.01 🎉

Comparison is base (4b520ac) 69.56% compared to head (287fc55) 69.57%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1141      +/-   ##
==========================================
+ Coverage   69.56%   69.57%   +0.01%     
==========================================
  Files          82       87       +5     
  Lines        3364     3444      +80     
==========================================
+ Hits         2340     2396      +56     
- Misses       1024     1048      +24     
Impacted Files Coverage Δ
src/core/models.py 95.71% <ø> (-0.49%) ⬇️
src/users/api/views.py 36.66% <36.66%> (ø)
src/pycontw2016/urls.py 84.21% <66.66%> (-1.51%) ⬇️
src/core/authentication.py 81.25% <79.31%> (-18.75%) ⬇️
src/attendee/api/views.py 42.85% <100.00%> (ø)
src/events/api/views.py 52.00% <100.00%> (ø)
src/sponsors/api/views.py 41.17% <100.00%> (ø)
src/users/api/urls.py 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

4 participants