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(board): support for combining tiles #1809

Merged
merged 12 commits into from
Feb 14, 2025
Merged

feat(board): support for combining tiles #1809

merged 12 commits into from
Feb 14, 2025

Conversation

emilielr
Copy link
Collaborator

@emilielr emilielr commented Feb 6, 2025

Mulighet for å kombinere stoppesteder til én


Motivasjon

Vi har fått flere henvendelser som handler om det er mulig å slå sammen stoppesteder til ett stoppested. Det er flere plasser hvor det ligger stoppesteder ganske nærme hverandre, og man ønsker å kunne slå disse sammen i stedet for å ha to separate tiles. I tillegg er det flere som har uttrykt at de ønsker å kunne se alt som går i en spesifikk retning, så vi har en hypotese om at ved å klare å kombinere flere plattformer så vil dette problemet også blir løst i større grad.

OBS: i første omgang er det kun mulig å enten kombinere alle stoppestedene i tavla, eller ingen, men det er lagt opp i databasen at man kan ha flere kombinerte tiles.

Endringer

  • Det skal nå være mulig å velge visningstype i innstillinger.
  • Databasen har fått et nytt felt: combinedTiles: [{ids: }] hvor man legger inn uuid for de tilene som skal kombineres. Velger man separat visning, så slettes dette feltet.
  • Det er ikke mulig å velge hvilke kolonner som skal vises i tavlevisningen om man har kombinert visning. Dette kan endres på senere, men er første iterasjon.
  • Lagt til måling på selve knappen. Vi skal også opprette en måling i Grafana.
  • Det er mulig å legge til gåavstand, men dette vil bare vises om visningen er separat.

image

image

image

Sjekkliste for Review

  • Sjekk at man får trykket på kombinert visning
  • Sjekk at man får trykket på separat visning
  • Sjekk at feltene i databasen oppdaterer seg deretter
  • Sjekk at det funker på gammel chrome
  • Sjekk at alle innstillinger fortsatt funker for hele tavla

@emilielr emilielr requested a review from purusott February 13, 2025 07:03
@emilielr emilielr merged commit 267f1d6 into main Feb 14, 2025
3 checks passed
@emilielr emilielr deleted the combine-tiles branch February 14, 2025 07:11
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.

Combine several bus stops in the same area as a single departure
2 participants