You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@atherdon I don't feel like it could be useful. I mean, if you want me to do it I will do it that's for sure, but here is the case (correct me if I am wrong):
Implement middlewares involves that every single request will pass through that middleware, or that is what I understand. This three helpers are been used only in certain points of the code, not every single request uses them so I think that implement them as middlewares will cause problem. Also, we probably will have to refactor some code to get it done right.
Again, maybe I am not understanding the middleware concept so if you think I'm wrong please report to me hahaha.
you're not wrong, but it depends.
I think i have a deal that will work for both of us.
At least let's update the filenames and make their calling at controllers
more clear.
Description/Steps to reproduce
We should plan an upgrade for this middleware helpers
Expected result
Additional information
The text was updated successfully, but these errors were encountered: