Feedback

[sumeetjain]

👍

But here are some things you can think about for taking it to the next level:

---

https://github.com/uanwar88/2015-02-16-message-board/blob/master/models/user.rb#L2

https://github.com/uanwar88/2015-02-16-message-board/blob/master/models/modules.rb#L13

Makes sense. But is there a way to infer the name of the table, so you don't have to set it manually?

---

https://github.com/uanwar88/2015-02-16-message-board/blob/master/models/user.rb#L16

So if I don't provide a username, what happens? If you needed to add additional validations later, how would you do that?

---

https://github.com/uanwar88/2015-02-16-message-board/blob/master/models/board.rb#L15-L19

Return value could be better; also documentation.

---

https://github.com/uanwar88/2015-02-16-message-board/blob/master/controllers/thread.rb#L3

What would it take for you to be able to run @thread.posts instead?

And could something similar be done for https://github.com/uanwar88/2015-02-16-message-board/blob/master/controllers/thread.rb#L4 ?

---

https://github.com/uanwar88/2015-02-16-message-board/blob/master/controllers/thread.rb#L11

What does this actually check for? Code comment would be good.