Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add auth extension for django-rest-knox #1163

Merged

Conversation

callumgare
Copy link
Contributor

Add a security schema extension for https://github.com/jazzband/django-rest-knox. Based off #264

It's my first time contributing and I've done my best to follow CONTRIBUTING.rst but I wasn't 100% about all the conventions with the tests so apologies it's it's a bit off.

@callumgare callumgare force-pushed the add-auth-extention-for-django-rest-knox branch from f97d945 to 624c9b9 Compare February 5, 2024 06:26
Copy link

codecov bot commented Feb 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (22b78d6) 98.58% compared to head (624c9b9) 98.59%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1163   +/-   ##
=======================================
  Coverage   98.58%   98.59%           
=======================================
  Files          72       74    +2     
  Lines        8771     8802   +31     
=======================================
+ Hits         8647     8678   +31     
  Misses        124      124           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tfranzel tfranzel merged commit 21c4e05 into tfranzel:master Feb 5, 2024
35 checks passed
@tfranzel
Copy link
Owner

tfranzel commented Feb 5, 2024

@callumgare excellent! Thank you very much for the PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants