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

Async support for the ViewSet class #8

Merged
merged 7 commits into from
Feb 27, 2023

Conversation

kartava
Copy link
Contributor

@kartava kartava commented Feb 26, 2023

This PR makes the following changes:

@kartava kartava changed the title Async Support for the ViewSet class #5 Async Support for the ViewSet class Feb 26, 2023
@kartava kartava changed the title Async Support for the ViewSet class Async support for the ViewSet class Feb 26, 2023
@kartava
Copy link
Contributor Author

kartava commented Feb 26, 2023

Hi @em1208, this PR is ready for review.

@em1208
Copy link
Owner

em1208 commented Feb 27, 2023

Hi @kartava, thanks a lot for your work on this. I will review your PR asap and once merged I will release a new version of the adrf package.

@kartava
Copy link
Contributor Author

kartava commented Feb 27, 2023

@em1208
Copy link
Owner

em1208 commented Feb 27, 2023

LGTM

@em1208 em1208 merged commit d3ba2a5 into em1208:main Feb 27, 2023
@Archmonger
Copy link

In the future I recommend squash merging to not clutter the commit history

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