philsof

This is good stuff, here are my notes on your code:

• Good job protecting against spoof posts by manually assigning the author_id to current_user.id and also restricting author_id from your post_params method. 👍 👍 👍
• Good work having if logged_in? logic on all of the appropriate views! 👍
• When searching by id number, instead of using find_by it is best to use find since that method is designed to search on the id field. [Here are the docs on find](http://guides.rubyonrails.org/active_record_querying.html#retrieving-a-single-ob

philsof

Good stuff, here are my notes on your code:

• You are getting this, good work! 👍

• This test does the job. Be aware though: Rspec comes with built-in expect change test capabilities, which can test if an object changes. (For example, if the length of an array increased by 1.) This will come in handy in the very near future.

• Note: You caught this error while I was writing this! Nice work! Here is what I wrote anyway:

This is a really tricky one to catch and is a good teaching moment: take a look at this test, which currently passes when it

philsof

Here are the docs and here is a highly rated StackOverflow answer. (The docs are pretty long, you will probably want to search the web page for "unique" to find the relevant info.)

Bottom line: You need to create an index and make the index unique:

add_index :table_name, :column_name, :unique => true

Hope this helps. Any questions let me know.

philsof

Migration table naming convention: groups_users

Migration file naming convention (the part of the file name that is not automatically generated): create_groups_users

Model class naming convention: GroupsUser

philsof

Useful Enumerable methods (can be called on Hash objects and Array objects):

• all?
• any?
• count
• each_slice
• each_with_index
• each_with_object
• find
• find_all
• find_index