search

Nat14 / Coffeeshop-group-project

Coffee shop study - LEARN Academy group project
0 stars 0 forks source link

Required fields for Meeting table #26

Open complinggirl opened 8 years ago

complinggirl commented 8 years ago

Its probably a good idea for some fields to be required. The confirm field should probably set to default "false". Currently, none of the fields are required or have defaults

Nat14 commented 8 years ago

The confirmed already set to false

complinggirl commented 8 years ago

Hey Nat, I don't think it can be set to false. When I test the model without specifying the confirm field,, the value of "meeting.confirm" is nil, not false. And I can't see in the model definition where the default is set to false. We can deal with it on Monday but there is certainly something bizarre that I get nil in the test. No worries. I just wanted to log the issue because my memory is appalling.