theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Auto compelte hm #79

Closed BasharDimaria closed 4 years ago

BasharDimaria commented 4 years ago

det finns lint bugs pga att det koden borde Andreas kolla på baserad på det som vi bestämmt idag , jag väntar på Feedbacken 💯 Jjag har stängat det föredånede autocomplete pull request , gör ett nytt istället som slutar på hm (hand made)

AndreasWJ commented 4 years ago

La till kommunikation mellan AddTrip/SearchTrip -> DropDown genom att lägga till en valueChange prop. valueChange kallas med det nya värdet i textfältet vid uppdateringar, antingen att användaren modifierar genom att skriva eller ta bort bokstäver, eller att välja en suggestion. valueChange kallar sedan setFrom och setTo.

Gjorde även en del ändringar för läsa koden enklare.

Var några problem med ESLint. Det stod: items: PropTypes.array.isRequired Vilket ESLint inte gillar då det inte är specifikt nog. För att fixa problem som dessa måste du skriva till fler detaljer, kunde lösa denna genom att ändra till: items: PropTypes.arrayOf(PropTypes.string).isRequired

@BasharDimaria kolla igenom min commit "Propagate value change to container and clean up code", säg till om det är något du undrar över. För övrigt, bra jobbat! 👍