search

citusdata / django-multitenant

Python/Django support for distributed multi-tenant databases like Postgres+Citus
MIT License
707 stars 116 forks source link

Use asgiref when available instead of thread locals (#176) #198

Closed gurkanindibay closed 9 months ago

codecov[bot] commented 9 months ago

Codecov Report

Merging #198 (e58a224) into main (30e3238) will not change coverage. Report is 1 commits behind head on main. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #198   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           44        44           
  Lines         1193      1238   +45     
=========================================
+ Hits          1193      1238   +45
Files Coverage Δ
django_multitenant/tests/settings.py 100.00% <100.00%> (ø)
django_multitenant/tests/test_models.py 100.00% <100.00%> (ø)
django_multitenant/tests/test_utils.py 100.00% <100.00%> (ø)

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

gurkanindibay commented 9 months ago

@darwing1210 Could you check this PR which fixes merge conflicts with the new changes

darwing1210 commented 9 months ago

@darwing1210 Could you check this PR which fixes merge conflicts with the new changes

Looks pretty well, thanks for fixing the merge conflicts

paddyobrien commented 6 months ago

@gurkanindibay any chance you could cut a new release with this in it? Trying to use this project in ASGI and running into issues that I think this change will fix

gurkanindibay commented 6 months ago

@paddyobrien Thanks for your interest in the project. We released 4.1.0 version. Thanks

paddyobrien commented 6 months ago

@gurkanindibay Thank you so much! ❤️