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

fix(board): use coordinates to get walking distance for multimodal coordinates #1686

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

adrianflatner
Copy link
Collaborator

@adrianflatner adrianflatner commented Oct 22, 2024

I dagens løsning støtter vi ikke ut å hente koordinater for multimodale stopp. Et multimodalt stopp er et stopp som har flere plattformer. F. eks er Oslo S og Jernbanetorget et multimodalt stopp. I spørringen i dag når man skal hente ut gåavstand, så henter man ikke ut koordinatene til multimodalen, men kordinatet til første quayen som ligger under multimodalen.

Bakgrunn: hvis man har en tavle satt opp i RG5, så viser det at gåavstanden til Dronningens gate er 4 minutter og til Oslo S 3 minutter. Dette er veldig rart da Oslo S er lengre unna enn Dronningens gate. Se slack her: https://entur.slack.com/archives/C9QDFPPQV/p1723182790373569 og her: Message in Slack.

Endringer:

  • Skriver om walkingDistanceQuery til å ta inn from og to som koordinater.
  • Legger til to nye spørringer: quayCoordinates og stopPlaceCoordinates.
    -> Disse to brukes for å hente ut koordinater til stoppestedet basert på om det er en quay eller stopPlace.

Note

Gjerne dobbeltsjekk at det stemmer for ditt favorittstoppested🛑

Copy link
Collaborator

@SelmaBergstrand SelmaBergstrand left a comment

Choose a reason for hiding this comment

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

Funka for alle stoppestedene jeg testa! 👏

Jeg ser vi kaster errors om fetchingen av walking distance ikke funker. Da vil vel brukeren redirectes til error-siden tror jeg. Er det det vi ønsker, eller vil vi heller vise en error-toast og la bruker fortsette? 🤔

Jeg approver likevel for det er veldig edge-case

@emilielr
Copy link
Collaborator

emilielr commented Oct 28, 2024

Jeg tenkte på det samme Selma, men det går på saveLocation()-spørringen, og har egentlig ikke så mye med logikken til å hente ut koordinater og riktig gåavstand. Jeg trodde egentlig dette var fikset i en tidligere PR @adrianflatner: #1684? 🤔 Tror noe her har overskrevet det slik at denne funksjonaliteten ikke har kommet med likevel..

@adrianflatner
Copy link
Collaborator Author

Jeg tenkte på det samme Selma, men det går på saveLocation()-spørringen, og har egentlig ikke så mye med logikken til å hente ut koordinater og riktig gåavstand. Jeg trodde egentlig dette var fikset i en tidligere PR @adrianflatner: #1684? 🤔 Tror noe her har overskrevet det slik at denne funksjonaliteten ikke har kommet med likevel..

Du har rett med at det er mer på saveLocation! I app/(admin)/edit/[id]/components/MetaSettings/Adress.tsx ligger den foreløpige logikken som catches. Men det skal nok endres litt mer på senere!

@adrianflatner adrianflatner merged commit 1306135 into master Oct 29, 2024
3 checks passed
@adrianflatner adrianflatner deleted the multimodal-coordinates branch October 29, 2024 07:33
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