Ruby on Rails Thursday, June 29, 2017

On 2017-Jun-29, at 10:59 , fugee ohu <fugee279@gmail.com> wrote:
On Wednesday, June 28, 2017 at 10:09:25 AM UTC-4, Rob Biedenharn wrote:
On 2017-Jun-28, at 01:51 , fugee ohu <fuge...@gmail.com> wrote:

On Monday, June 26, 2017 at 9:43:19 AM UTC-4, Rob Biedenharn wrote:
At this point, you should be able to look around a small codebase like this and figure it out, but here are the "hints":





If you don't get how this is working, point to a specific thing and ask a more directed question.

-Rob

P.S. The routes could easily be amended to be:
resources :comments, except: [:new]

On 2017-Jun-25, at 22:04 , fugee ohu <fuge...@gmail.com> wrote:

I'm looking at the Dustin Fisher example app on github https://github.com/DustinFisher/acts-as-commentable-with-threading-example/tree/master/ but his comments_controller.rb has no new action Can someone clarify please?


I put a new action in the comments controller since you don't use a link to new you render the form instead

Unless you're changing other parts, too. You will NEVER hit the :new action because you ALWAYS create the comment form in the context of a Post or another Comment. Like I said, you could add `, except: [:new]` to the route if it bothers you.
-Rob


 So in your example the forms already on the page in index view instead of a link to new comment? If yes whats wrong with a link to new comment instead of a text area and submit button already rendered?

If you do that, you need to pass the "parent" object to the `new` action. There's nothing "wrong" with that. You probably need to define your own new_comment route if you want the URL to look nice and not have query parameters to define the parent object. (If you intend to eliminate threaded comments where the parent is another comment, you have just a little more work to do, but then you might not even need Comment to be polymorphic [if you can't comment on more than one kind of model].)

-Rob

No comments:

Post a Comment