need to work on form validation