-
-
Notifications
You must be signed in to change notification settings - Fork 158
Fix missing HttpContext on ValidationContext during atomic operations #1251
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
Fix missing HttpContext on ValidationContext during atomic operations #1251
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1251 +/- ##
=======================================
Coverage 92.62% 92.62%
=======================================
Files 243 243
Lines 7795 7796 +1
=======================================
+ Hits 7220 7221 +1
Misses 575 575
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Hi @Tommy228, thanks for your contribution! I intend to look at it somewhere this week. |
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.
Thanks for carefully adhering to our code style and naming conventions. See comments below.
Closes #1250
This PR
HttpContext
to theValidationContext
when validating objects during atomic operations, allowing dependency injectionQUALITY CHECKLIST