- Notifications
You must be signed in to change notification settings - Fork 231
Add drf Request support #1117
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
Add drf Request support #1117
Conversation
💚 CLA has been signed |
💔 Build Failed
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪Steps errors Expand to view the steps failures
|
Test | Results |
---|---|
Failed | 0 |
Passed | 7344 |
Skipped | 7899 |
Total | 15243 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch and nice fix! Just a small nitpick about naming :)
Co-authored-by: Benjamin Wohlwend <bw@piquadrat.ch>
Co-authored-by: Benjamin Wohlwend <bw@piquadrat.ch>
@Niccolum if you have a second, could you have a look at our CLA and sign it if you agree? That'll allow me to merge your PR :) |
@beniwohli, I signed it. Hope, I did it correct ) |
@Niccolum sorry about the hassle, but could you sign the CLA again using the email you used for the commits? The one matching |
@beniwohli Oh, sorry for this |
@Niccolum yes, perfect, thanks! I'll merge the PR in a bit! |
What does this pull request do?
Related issues
closes #ISSUE