kaushalwagle / blog4492 Goto Github PK

Kaushal,

So far this is looking good. I pulled the commit from 2/2/18 around 4:00 to look at. Looking at your code it seems like the contact form should function properly so that is good. I did leave some comments about convention in other issues as well. You have your blog controller created but it doesn't look like you were able to get the blog domain and saving the form implemented. Keep up the good work and let me know if you have any questions.
-Brandon

Right here you are setting the values of your contact object. This could be done this way, however, if you remember to give your html element inputs the same name as your domain's properties, Grails will handle mapping those properties behind the scenes. For example if you give your domain a firstName and lastName property, and give your fields name="firstName" and another name="lastName", then submit your form you won't have to explicitly set contact.firstName = params.firstName. Let me know if that doesn't make sense.

You are rendering index.gsp and aboutUs.gsp here. No need to add .gsp at the end. :)

Probably don't want this to be nullable since this field should always be set when it's created. These are always good fields to set when the object is instantiated because it's usually going to be = to new Date() (which is today).