[ ] A note about issues/feedback. You add an "X" to check off the checkboxes to indicate you have completed the work or responded to the questions. When all done, then you can close the issue. See example:
[x] A completed task.
[x] Is the database design document up-to-date? I think you were looking at some lookup table for category, for example?
[ ] Something to keep in mind in general - if there were an error in the userDao, what would the user experience be? You might consider throwing an exception (you can create your own exceptions) that the controller would catch and then route the user to a friendly-looking error page.
[ ] There is some hard-coded data here. Could that be handled in a unit test instead?
[x] Good start on your unit test for User. The add assertions look very thorough! Keep up that same quality on the the remaining tests in that class.
[ ] Is there a class diagram? This is checked off in your project plan, but I didn't see it anywhere?