Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

URL med GET parameter #136

Open
binderup opened this issue Jan 31, 2023 · 8 comments
Open

URL med GET parameter #136

binderup opened this issue Jan 31, 2023 · 8 comments

Comments

@binderup
Copy link

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.

@rndfm
Copy link
Owner

rndfm commented Jan 31, 2023

Ja. Der har faktisk været et ønske om det tidligere.
#28

Men elspotpris.dk burde ikke glemme indstillingerne med mindre man aktivt går ind på enheden og rydder dataen.

@aswerkljh
Copy link

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.

@MrAdam
Copy link

MrAdam commented Apr 13, 2023

@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:

  1. Indstillinger fra query parameters bliver loaded ved page-load og overskriver hvad end der er lagret i state (local storage).
  2. 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
Copy link
Owner

rndfm commented Apr 14, 2023

@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:

  1. Indstillinger fra query parameters bliver loaded ved page-load og overskriver hvad end der er lagret i state (local storage).
  2. 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.

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 :)

@rndfm
Copy link
Owner

rndfm commented Apr 14, 2023

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 :)

@MrAdam
Copy link

MrAdam commented Apr 14, 2023

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.

@rndfm
Copy link
Owner

rndfm commented Apr 14, 2023

Fedt! Mange tak!

Jeg kan evt. godt stå for UI delen hvis det er?

@asiudoisjdfo
Copy link

Hello guys - any news on this one? :-) If I can help in any way, let me know :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants