Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(demo): possible to try tavla without creating a user #1569

Merged
merged 15 commits into from
Jul 9, 2024
Merged

Conversation

emilielr
Copy link
Collaborator

@emilielr emilielr commented Jul 3, 2024

  • Use local storage to store the demo board. Created a custom hook to keep track of and update the local storage, so it can be used by other components if needed in the future -> if demo board is no longer supposed to be in local storage (e g moved to firestore), the hook can be omitted
  • still need to pass the demoboard prop down to the tilecard to listen for changes in the tiles. not optimal, but the only solution i found to be able to update local storage (it doesnt seem there is a way to listen to changes in the local storage)
  • if demo board, show another error text on "gåavstand" (see pic 2)

image
image

@emilielr emilielr requested review from oyvindgrutle and purusott and removed request for lindtvedtsebastian July 3, 2024 12:20
@purusott
Copy link
Collaborator

purusott commented Jul 5, 2024

noen tanker:

  • Jeg lurer på om det med gåavstand kan være forstyrrende for brukere. Per nå funker den ikke med mindre man også har lokasjon. Bør vi ha den default på (men skjule switchen for brukeren) så man ser gåikonet, eller fjerne det f.eks 💭
  • Hvis man er på demosiden og logger inn, vil man komme inn på demosiden med navbaren
  • image
  • Kan vi sette opp en posthog måling når brukere oppretter en tavle på denne siden elns? da kan vi måle litt effekt 📈 🎆

Copy link
Collaborator

@purusott purusott left a comment

Choose a reason for hiding this comment

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

👏 legger vi inn noe måling her er det good for me ⭐

@emilielr
Copy link
Collaborator Author

emilielr commented Jul 8, 2024

Vi velger å gå for at gåavstand bare ligger der slik den gjør inntil videre. Jeg har lagt til måling på opprett-bruker knappene og legg til stoppested knappen (for å sjekke om noen faktisk bruker den). Man blir også redirectet til "tavler"-siden om man logger inn direkte fra demo-siden.

@emilielr emilielr merged commit 9dac2c5 into master Jul 9, 2024
3 checks passed
@emilielr emilielr deleted the demo-tavla branch July 9, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants