-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Migrate formatting middlewares to Method
formatters
#1209
Conversation
0709f77
to
421950a
Compare
5f60251
to
b97db6a
Compare
waiting for #1166 to land before looking this over. |
b97db6a
to
aa5e6f3
Compare
aa5e6f3
to
c9bc02b
Compare
(pythonic_middleware, 'pythonic'), # Delete | ||
(normalize_errors_middleware, 'normalize_errors'), # Add async | ||
(validation_middleware, 'validation'), # Add async | ||
(abi_middleware, 'abi'), # Delete | ||
] |
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.
These comments are notes to myself. Will remove.
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.
@dylanjw I am planning on taking this over shortly if you don't have objections! When I was looking this over, it looks like the attrdict_middleware didn't get moved over. I assume that the plan is to shift it still? Or did you run into something where it didn't make sense any more?
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.
My plan at this time was to move the attrdict_middleware functionality to the method formatters later. I didnt run into any complications.
I broke out the commit re #1197. I will put it in a pull request once this is merged. |
This is ready for review. |
Moved these commits over to #1461. Closing here. |
What was wrong?
Based on: #1166
Related to Issue #1161 and #1057
TODO:
validation_middlewareCute Animal Picture