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

fix #867 by adding keyword argument params #917

Closed
wants to merge 1 commit into from
Closed

fix #867 by adding keyword argument params #917

wants to merge 1 commit into from

Conversation

michaelrigart
Copy link

Stops triggering the deprecation warning: ActionController::TestCase HTTP request methods will accept only keyword arguments in future Rails versions.

@mcmire
Copy link
Collaborator

mcmire commented Jun 14, 2016

Which Rails version are you getting this on?

@mcmire
Copy link
Collaborator

mcmire commented Jun 14, 2016

Oh, gotcha, this is Rails 5, thanks.

@mcmire mcmire mentioned this pull request Sep 4, 2016
9 tasks
mygulamali added a commit that referenced this pull request Sep 23, 2016
Rails 5 has deprecated the keywords arguments for #permit. This patch
scopes the request params according to the version of Rails.

Fixes #867.  Supercedes #917 with passing tests.

Thank you to @freesteph for helpin' out!
mygulamali added a commit that referenced this pull request Sep 23, 2016
Rails 5 has deprecated the keywords arguments for #permit. This patch
scopes the request params according to the ActionPack version.

Fixes #867.  Supercedes #917 with passing tests.

Thank you to @freesteph for helpin' out!
@mcmire mcmire added this to the v4.0 milestone Sep 19, 2017
@mcmire
Copy link
Collaborator

mcmire commented Sep 30, 2017

This was fixed by ce9624b.

@mcmire mcmire closed this Sep 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants