search

ArangoDB-Community / ArangoBnB

16 stars 3 forks source link

Simplify react markup of Home/Landing page #36

Closed couds closed 3 years ago

couds commented 3 years ago

Simplify Home markup creating a new DestinationPicker component to encapsulate all the logic inside.

Also add Travelers picker

cw00dw0rd commented 3 years ago

@lostpebble could you review, please?

lostpebble commented 3 years ago

All looks good @couds - I also noticed that the second date input has a placeholder that says "to this dates" - just need to take the "s" off there.

Also, it seems I am able to select an earlier date than the "from date" in the "to date" input- some validation we should handle is to make that at least a day past the first date.

cw00dw0rd commented 3 years ago

Can this be merged?

couds commented 3 years ago

I need to fix some typos, I will do it today or tomorrow

couds commented 3 years ago

@cw00dw0rd you can merge it now =) Thanks