Closed dixpac closed 4 months ago
Great change @dixpac! Thanks for tackling.
Left one comment. I think once we address that and add a test for this we'll be ready to merge.
@josefarias added stringify_keys
and a simple test
Awesome. Thanks for tackling @dixpac!
Before, default
src
was set usingrequest.path
which returns virtual path without query string.This PR changes default implementation to use
request.fullpath
which returns virtual path with query string.Form example; In our app we have one controller endpoint where we filter the results not just base on user entry
q
but also query strings.