Closed ITJohan closed 4 years ago
Fina grejer, har inte mycket att kommentera på. Bra jobbat! Och bra addition av border-radius och button padding i vår theme.
Det enda jag märkte var att API url:en hade ett frågetecken i det fallet då man inte valde några filter. Se bilden nedan:
Det är inga problem med det just nu, men tänker utan frågetecken hade varit rätt egentligen. Vid en GET så använder man ju bara frågetecken i det fall man vill specificera filters. Bra att ha där ifall koden på backend ändras, så har vi skrivit rätt url oavsett 😄.
Lägger till en egen commit som fixar seat select elementet, vet inte vad som hänt där men märkte det nu.
Bra noterat! Ändrade nu så den bara är med när man söker något:)
Saker jag fixat i denna PR: