-
Notifications
You must be signed in to change notification settings - Fork 67
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
feature: added management command for simple use of create_public_tenant #565
feature: added management command for simple use of create_public_tenant #565
Conversation
@Dresdn tagging you since I can't assign a reviewer. We added this command locally to our project and I figured it would be a good contribution. |
Thanks @jgentil! Appreciate the contribution. |
@jgentil FYI that the tests are failing, but I did take this opportunity to update all dependencies to latest supported ones |
Ah I need to fix the f string to support Python 3.7. |
I just cleaned up the dependencies of EOL versions, so 3.7 isn't an issue anymore. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #565 +/- ##
==========================================
+ Coverage 88.36% 89.05% +0.68%
==========================================
Files 18 20 +2
Lines 619 658 +39
Branches 116 118 +2
==========================================
+ Hits 547 586 +39
Misses 62 62
Partials 10 10 ☔ View full report in Codecov by Sentry. |
Thanks! I'll do a release here, too, so we can get this and Django 5.0 support out the door. |
Oh. Haha oh well, I guess that works too then! |
I'm helping!
Checklist
CHANGELOG.md
Pull Request type
Please check the type of change your PR introduces:
Related issue(s)