Skip to content

RO-1872: Lese gamle url-parametre som brukes i regobs.no #729

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

Merged
merged 6 commits into from
Apr 25, 2025

Conversation

gruble
Copy link
Contributor

@gruble gruble commented Apr 3, 2025

Se saken:
https://nveprojects.atlassian.net/browse/RO-1872

Vi støtter nå de fleste URL-parametere som regobs.no bruker

  • NWLat, NWLon, SELat, SELon
  • GeoHazards
  • SelectedNumberOfDays
  • FromDate og ToDate
  • ObserverNickName
  • SelectedRegion
  • Countries (fjernes ved oppstart siden vi ikke støtter det på nye regobs)
  • SelectedRegistrationTypes
  • ObserverCompetence

Forenklet også test-oppsettet litt.

Copy link

github-actions bot commented Apr 3, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-729.westeurope.azurestaticapps.net

@gruble gruble changed the title RO-1872: Lese gamle regobs url parametre RO-1872: Lese gamle url-parametre som brukes i regobs.no Apr 3, 2025
@amish1188
Copy link
Contributor

Etter testing med å kopiere URL fra Regobs ser jeg at koordinatene settes riktig i URLen i PRen din. Men de vises på den gamle Regobs måten, altså NWLat, NWLon, SELat, SELon, i stedet for nwLat, nwLon, seLat, seLon. Kartutsnittet vises ikke riktig. Jeg la også merke til at når jeg limer inn en URL fra Regobs med koordinater, blir koordinatene aldri oppdatert i PR-en når jeg panorerer i kartet. Er dette med vilje?

@gruble
Copy link
Contributor Author

gruble commented Apr 3, 2025

Etter testing med å kopiere URL fra Regobs ser jeg at koordinatene settes riktig i URLen i PRen din. Men de vises på den gamle Regobs måten, altså NWLat, NWLon, SELat, SELon, i stedet for nwLat, nwLon, seLat, seLon. Kartutsnittet vises ikke riktig. Jeg la også merke til at når jeg limer inn en URL fra Regobs med koordinater, blir koordinatene aldri oppdatert i PR-en når jeg panorerer i kartet. Er dette med vilje?

Takk for at du testet dette skikkelig!
Jeg har kun laget enhetstester, og dette var åpenbart ikke godt nok.
Det at kartutsnitt ikke vises riktig har jeg registrert en separat feil på. Dette virker heller ikke på beta.
Vi må også fjerne de gamle URL-parametrene fra URL'en etter vi har lest dem.
Jeg gjør om denne PR'en til kladd, siden dette var vanskeligere å fikse enn jeg trodde. Så kan vi heller fortsette med dette når vi får tid. Jeg venter egentlig på en avklaring fra Jostein på noe.

@gruble gruble marked this pull request as draft April 3, 2025 13:09
@jorgkv jorgkv self-assigned this Apr 23, 2025
@jorgkv jorgkv force-pushed the feature/ro-1872-lese-gamle-regobs-url-parametre branch from 8f4b6da to 82fe83e Compare April 24, 2025 08:12
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-729.westeurope.azurestaticapps.net

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-729.westeurope.azurestaticapps.net

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-729.westeurope.azurestaticapps.net

@jorgkv jorgkv marked this pull request as ready for review April 24, 2025 09:13
@jorgkv jorgkv requested a review from amish1188 April 24, 2025 09:13
Copy link
Contributor

@amish1188 amish1188 left a comment

Choose a reason for hiding this comment

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

virker greit. bra du fikset problemet med koordinater

@jorgkv jorgkv merged commit 25315dd into develop Apr 25, 2025
8 checks passed
@jorgkv jorgkv deleted the feature/ro-1872-lese-gamle-regobs-url-parametre branch April 25, 2025 09:34
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