-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Deprecated route_xxx methods for route_params and route_? declared. #1344
Comments
cc: @namusyaka, maybe you have a suggestion? |
@dblock Hmm, I think the |
If you have time @namusyaka, the simplest thing to look at is https://github.com/dblock/grape-swagger/tree/fix-deprecations, try to fix the remaining deprecations. I think the simplest would be to just expose |
For now I am monkey-patching the deprecation to avoid too many warnings being spewed. module Grape
class Router
class Route
private
# TODO: remove when https://github.com/ruby-grape/grape-swagger/pull/368 and https://github.com/newrelic/rpm/pull/235 are finished
def warn_route_methods(name, location, expected = nil)
end
end
end
end |
Closing via #1357. |
I've PRed ruby-grape/grape-swagger#368, but it has two problems, access to
route_params
androute_hidden
in a dummy API declared asGET '/foobar', hidden: true
.The
route_xxx
deprecated code reaches into@options
, which isn't@settings
, and that is no longer exposed. So.params
is not the same thing as the oldroute_params
. What should we do about this one?The text was updated successfully, but these errors were encountered: