-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Use a single term instead of all terms used to describe path and URL helpers together #36254
Conversation
03c8e0d
to
e9b2e64
Compare
e9b2e64
to
797e740
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"named route helpers" is the correct name for this
Thanks, I'll use that term. |
967b2e7
to
93bc6d2
Compare
I've pushed the changes. |
guides/source/routing.md
Outdated
@@ -210,7 +210,7 @@ end | |||
|
|||
This will create a number of routes for each of the `articles` and `comments` controller. For `Admin::ArticlesController`, Rails will create: | |||
|
|||
| HTTP Verb | Path | Controller#Action | Named Helper | | |||
| HTTP Verb | Path | Controller#Action | Route Helper | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not it be Named Route Helper
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes 😢
Fixed now.
93bc6d2
to
208a861
Compare
208a861
to
18504eb
Compare
Use a single term instead of all terms used to describe path and URL helpers together
Fixes #36233.
I agree with the sentiment behind the original issue: We need a single term to say "URL and Route helpers" and this is my attempt at getting there.
I'm open to suggestions for a better term if anyone has any.