Skip to content
This repository was archived by the owner on Aug 17, 2017. It is now read-only.

Change readme code examples to be more explicit#152

Open
scottillogical wants to merge 1 commit intorails:masterfrom
scottillogical:master
Open

Change readme code examples to be more explicit#152
scottillogical wants to merge 1 commit intorails:masterfrom
scottillogical:master

Conversation

@scottillogical
Copy link
Copy Markdown

Updating readme to show the two different examples more explicitly. The previous code example was confusing to me.

Updating readme to show the two different examples more explicitly.  The previous code example was confusing to me.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be ActionController::ParameterMissing instead.

I believe mine #158 can be closed when your pull request is merged.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Language could, perhaps, be a little less flowery.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants