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

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

Merged
merged 5 commits into from
Sep 26, 2023

Conversation

gurkanindibay
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 26, 2023

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%> (ø)

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

@gurkanindibay gurkanindibay marked this pull request as ready for review September 26, 2023 09:35
@gurkanindibay gurkanindibay changed the title News amp asgi support Use asgiref when available instead of thread locals (#176) Sep 26, 2023
@gurkanindibay
Copy link
Contributor Author

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

@darwing1210
Copy link

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

Looks pretty well, thanks for fixing the merge conflicts

@gurkanindibay gurkanindibay merged commit fd84aee into main Sep 26, 2023
53 checks passed
@gurkanindibay gurkanindibay deleted the NewsAmp-asgi-support branch September 26, 2023 15:30
@paddyobrien
Copy link

@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
Copy link
Contributor Author

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

@paddyobrien
Copy link

@gurkanindibay Thank you so much! ❤️

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

Successfully merging this pull request may close these issues.

3 participants