-
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
[ci skip] Fix an object of reference
#19483
Conversation
@@ -1510,7 +1510,7 @@ comments on articles. | |||
|
|||
We're going to see the same generator that we used before when creating | |||
the `Article` model. This time we'll create a `Comment` model to hold | |||
reference of article comments. Run this command in your terminal: | |||
reference to an article. Run this command in your terminal: |
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.
I think I like better the way it was before.
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.
In rails guide about associations, I think reference
has special meaning. reference
is equivalent to belongs_to
. So I feel
Comment
model to hold reference of article comments
means Comment
model belongs_to article comments.
If this sentence means that creating Comment
model to persist article comments, maybe it is better to write that ... create a Comment
model to hold article comments.
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.
I think it's fine to say reference here. For me the use of the word reference here isn't special (no backticks), but sets up the reference coming in migration nicely.
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.
LGTM
[ci skip] Fix an object of `reference`
thanks for your review ❤️ |
Fix original docs via rails/rails#19483 :)
Comment
has reference to an article, orCommand
has a comment of an article.In this context
reference
should be highlighted.