add basic validtion