thuquach/ask_me — pull request #1 by evenellie (2013-07-27 18:28:56+00:00)
Title: Pending tests deleted and links to home and discussion added to views.

Hi Tu,

I appreciated that you named your model discussions! I named mine question and it has a question attribute, but that is confusing and I wish I thought to use discussion for the model name.

The functionality that you added is working; I started up the app and tried it out in the browser. I deleted some empty (pending) tests and added a link to the discussions index at the top of the application view and a back to the Discussion for the response show page.

It looks like you aren't done with this app yet?. I see you that you added "has_and_belongs_to_many :responses" to the user model, but it doesn't look like the responses model "has_and_belongs_to_many :users", and I don't see the join table in your schema. Your app is tracking the user that creates a response, but not users that are editing responses or the responses that a particular user has created or edited. Let me know if I can help clarify things.

Ellen