search

theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Trips pagination #67

Closed ITJohan closed 4 years ago

ITJohan commented 4 years ago

Denna PR adderar pagination till trips. För att få detta att fungera så valde jag att refaktorera hämtningen av trips från Display och SearchTrips till Trips eftersom det är lättare att hantera nuvarande trips på ett ställe.

Edit: Det verkar som att det är problem med backend för tillfället så den fetchar inte trips. Detta gör det svårt att se pagination eftersom den inte visas när antal trips är 0.

AndreasWJ commented 4 years ago

Som du nämnde så verkar det vara något fel på backends SSL certifikat. Kan approva nu om du vill men hade varit bra för mig att kika igenom funktionen 🙃

ITJohan commented 4 years ago

Ja, let's wait :grin:

AndreasWJ commented 4 years ago

Låter dig merga, om du vill göra någon ändring eller nåt 👍

ITJohan commented 4 years ago

Tack för bra feedback! Tänkte också lite på det med om man ska gömma pagination vid en sida. Kom fram till samma slutsats att det kanske är bra om den ligger där som standard även vid en sida så att användaren känner igen sig. Kan ju köra på det så länge. :)

Håller med om det med att man borde skickas till toppen när man byter sida! Slänger upp en fix på det och mergar sen. :smiley: