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

URL med GET parameter #161

Closed
wants to merge 8 commits into from
Closed

URL med GET parameter #161

wants to merge 8 commits into from

Conversation

MrAdam
Copy link

@MrAdam MrAdam commented Apr 13, 2023

Jeg har forsøgt at fikse #136 og #28.

Som jeg har beskrevet i #136 så ser jeg to mulige løsninger, og det her er så mit forsøg med modellen hvor de indstillinger der kan gemmes i query parameter kun eksisterer dér, og ikke også i local storage.

Jeg har ikke kodet Svelte før, så bær over med mig hvis det ikke er godt nok - jeg er åben for input 😄

Copy link
Owner

@rndfm rndfm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hvorfor må customConsumption ikke være med? :)

@rndfm
Copy link
Owner

rndfm commented Apr 14, 2023

Beklager jeg ikke har været hurtig nok ude inden du har lavet løsning 2. Men jeg tror virkelig det er løsning 1 vi skal gå med.

Eller måske en kombination så man stadig kunne nappe url'en fra browseren.
Men localstorage er vel et must således at alle indstillinger ikke bare er standard når man tilgår "roden" uden get parametre.

@rndfm
Copy link
Owner

rndfm commented Apr 14, 2023

Vi kommer også i problemer med SEO når google begynder at indekserer tusinde af forskellige URL'er. Så måske ikke en kombination. Men måske siden skulle redirect efter den har sat indstillingerne fra get parametrene således at der ikke eksisterer sider på mange mange forskellige URL'er.

@MrAdam
Copy link
Author

MrAdam commented Apr 14, 2023

Jeg er helt enig 😄
Jeg lukker det her og gør mig et forsøg med løsning 1. snarest.

@MrAdam MrAdam closed this Apr 14, 2023
@MrAdam MrAdam deleted the 136-URL-med-GET-parameter branch April 20, 2023 13:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants