rndfm
URL med GET parameter
Skulle i dag forklare et ældre familiemedlem hvorfor det var at elspotpriser af og til glemmer netselskab mv. - Det ville gøre delbarheden til andre nemmere hvis alle tilvalg på hjemmesidens formular var tilgængelige via url'ens get parametre.
Ja. Der har faktisk været et ønske om det tidligere. https://github.com/rndfm/elspotpris/issues/28
Men elspotpris.dk burde ikke glemme indstillingerne med mindre man aktivt går ind på enheden og rydder dataen.
Men elspotpris.dk burde ikke glemme indstillingerne med mindre man aktivt går ind på enheden og rydder dataen. Det er dog ikke alle der tillader cookies by default, så GET parameters ville være nice.
@rndfm, jeg kiggede lidt på at implementere det i aftes, men er lidt i tvivl om hvordan det håndteres smartest.
Som jeg ser det er der to tilgange:
- Indstillinger fra query parameters bliver loaded ved page-load og overskriver hvad end der er lagret i state (local storage).
- Indstillinger bliver kun gemt i query parameters.
Jeg har implementeret model 2. da jeg tænker det giver bedre usability at man bare kan ændre sine indstillinger, og så kopiere nuværende URL og sende til nogen, og så ser det ens ud hos dem. Problemet med den model er bare at jeg har været nødt til at implementere en beforeNavigate hook som kopierer alle query parameters fra nuværende url ind på to url for at sikre at indstillinger er preserved på tværs af page navigation (fra "/" til "/live" fx.) hvilket jeg ikke synes er en pæn løsning.
Men hvis den løsning kan bruges, kan jeg få lavet mit PR færdig snart så vi kan få det ind.
@rndfm, jeg kiggede lidt på at implementere det i aftes, men er lidt i tvivl om hvordan det håndteres smartest.
Som jeg ser det er der to tilgange:
- Indstillinger fra query parameters bliver loaded ved page-load og overskriver hvad end der er lagret i state (local storage).
- Indstillinger bliver kun gemt i query parameters.
Jeg har implementeret model 2. da jeg tænker det giver bedre usability at man bare kan ændre sine indstillinger, og så kopiere nuværende URL og sende til nogen, og så ser det ens ud hos dem. Problemet med den model er bare at jeg har været nødt til at implementere en
beforeNavigatehook som kopierer alle query parameters fra nuværende url ind påtourl for at sikre at indstillinger er preserved på tværs af page navigation (fra "/" til "/live" fx.) hvilket jeg ikke synes er en pæn løsning.Men hvis den løsning kan bruges, kan jeg få lavet mit PR færdig snart så vi kan få det ind.
Hej! Tak fordi du kigger på denne :)
I mit hoved er det løsning 1. jeg har tænkt på.
Løsning 2. tror jeg bliver lidt kedelig for alle dem som i forvejen bruger siden og bare går til https://elspotpris.dk. De ville alle skulle vælge deres indstillinger igen og så kun tilgå url'en med alle query parametrene.
Løsning 1 kræver dog lidt UI da man så skal have en mulighed for at lave et link ud fra ens indstillinger. Men det tænker jeg ikke er den helt store omgang :)
Og løsning 1 giver faktisk stadig mulighed for at bruge siden som løsning 2 hvor man bare altid tilgår siden med GET parametre. Burde vel også virke uden cookies :)
Jeg laver nyt PR snarest og kigger på at få implementeret løsning 1. Derudover tænker jeg at UI-delen kan laves i et separat PR bagefter.
Hello guys - any news on this one? :-) If I can help in any way, let me know :-)
