Skip to content
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

Support for verb/query params with new framework setting... #385

Open
charlesr1971 opened this issue Jan 22, 2019 · 1 comment
Open

Support for verb/query params with new framework setting... #385

charlesr1971 opened this issue Jan 22, 2019 · 1 comment

Comments

@charlesr1971
Copy link

I think it would be useful to have a setting called:

variables.framework.queryParamVerbMap

This would allow one or more verbs to allow query params to be passed into methods as optional arguments. Like:

variables.framework.queryParamVerbMap = {
    put: true,
    delete: true,
    post: true
}

The GET verb implicitly applies this rule, already, but the other 3 verbs should also be able to transmit query params in the API URL.

@atuttle
Copy link
Owner

atuttle commented Oct 1, 2020

I think I'd rather seem them always allowed in all cases. We already have a cascading priority setup, so I don't see any reason not to do the same for all verbs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants