You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2018/08/22 19:30:11 UTC

[GitHub] dangogh commented on issue #2635: Fixes #2397 - Removed NOT NULL constraints on 'value' from parameters…

dangogh commented on issue #2635: Fixes #2397 - Removed NOT NULL constraints on 'value' from parameters…
URL: https://github.com/apache/trafficcontrol/pull/2635#issuecomment-415151636
 
 
   @mitchell852 -- I agree with @rob05c on this one -- removing "required" validation on value in the go code would take care of this.  The column in the db should be NOT NULL with a default of empty string.  This eliminates the need to distinguish between empty string and null in code.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services