Compare commits
No commits in common. "7bcb09a15cb883d123f84886766b8b89753b56eb" and "1c84dbd4f641250345e6fa1138722fa0cd1ab530" have entirely different histories.
7bcb09a15c
...
1c84dbd4f6
3 changed files with 30 additions and 86 deletions
32
CHANGELOG.md
32
CHANGELOG.md
|
|
@ -9470,35 +9470,3 @@ Neste steg:
|
||||||
**Rullet ut umiddelbart** (produksjonsbug som traff ALLE brukere som
|
**Rullet ut umiddelbart** (produksjonsbug som traff ALLE brukere som
|
||||||
prøvde kart-basert måling) -- kun frontend-image, `teecup_api`
|
prøvde kart-basert måling) -- kun frontend-image, `teecup_api`
|
||||||
bekreftet uendret.
|
bekreftet uendret.
|
||||||
|
|
||||||
52. **PRODUKSJONSBUG: kartet sentrerte alltid på en hardkodet Oslo-
|
|
||||||
koordinat, aldri brukerens faktiske posisjon — 2026-08-08, umiddelbar
|
|
||||||
brukeroppfølging etter punkt 51.** Bruker: "Kartet viser ikke hvor
|
|
||||||
jeg faktisk er, men en eller annen vilkårlig by."
|
|
||||||
|
|
||||||
**Rotårsak:** `MapPointPicker` sin `center: [10.7522, 59.9139]` var
|
|
||||||
en HARDKODET Oslo-koordinat, ledsaget av en kommentar som hevdet
|
|
||||||
"real app centers on last-known position" -- den logikken fantes
|
|
||||||
ALDRI, kun påstanden i kommentaren. Kartet åpnet dermed alltid på
|
|
||||||
samme faste punkt i Oslo, uansett hvor i verden (eller landet)
|
|
||||||
brukeren faktisk befant seg.
|
|
||||||
|
|
||||||
**Rettet:** kartet henter nå brukerens ekte GPS-posisjon (ett
|
|
||||||
`getCurrentPosition`-kall, samme mønster som "Min posisjon nå"-veien
|
|
||||||
-- ingen løpende `watchPosition`) FØR selve Mapbox-kartet
|
|
||||||
initialiseres, og sentrerer der. Oslo-koordinaten beholdt KUN som
|
|
||||||
fallback for det tilfellet posisjon ikke kan hentes (avslått
|
|
||||||
tillatelse, tidsavbrudd, ingen støtte i nettleseren).
|
|
||||||
|
|
||||||
`tsc --noEmit` rent. **Scratch-verifisert i ekte nettleser** (tjueandre
|
|
||||||
scratch-miljø denne økten, samme migrasjon-002-forsiktighet,
|
|
||||||
`teecup_db` bekreftet uendret): overstyrte `navigator.geolocation` til
|
|
||||||
Bergen sentrum (60,39°N 5,32°Ø) og bekreftet kartet faktisk åpnet der
|
|
||||||
(synlig annen bystruktur enn Oslo-testen fra punkt 51, pluss Mapbox
|
|
||||||
sin egen posisjons-markør synlig i visningen) i stedet for på den
|
|
||||||
tidligere faste Oslo-koordinaten. Scratch-miljøet ryddet opp
|
|
||||||
fullstendig.
|
|
||||||
|
|
||||||
**Rullet ut umiddelbart** (samme produksjonsbug-alvorlighet som
|
|
||||||
punkt 51 -- traff alle som brukte kart-valget utenfor Oslo sentrum)
|
|
||||||
-- kun frontend-image, `teecup_api` uendret.
|
|
||||||
|
|
|
||||||
|
|
@ -36,65 +36,41 @@ export function MapPointPicker({ onConfirm }: { onConfirm: (lngLat: LngLat) => v
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
let cancelled = false
|
mapboxgl.accessToken = token
|
||||||
|
const map = new mapboxgl.Map({
|
||||||
|
container,
|
||||||
|
style: "mapbox://styles/mapbox/satellite-v9",
|
||||||
|
center: [10.7522, 59.9139], // Oslo fallback; real app centers on last-known position
|
||||||
|
zoom: 16,
|
||||||
|
attributionControl: false,
|
||||||
|
})
|
||||||
|
mapRef.current = map
|
||||||
|
|
||||||
// Reell bug funnet 2026-08-08: kartet sentrerte ALLTID på en hardkodet
|
map.on("load", () => setLoaded(true))
|
||||||
// Oslo-koordinat -- en kommentar her hevdet "real app centers on
|
map.on("error", () => setError("Kunne ikke laste kartet."))
|
||||||
// last-known position", men den logikken fantes aldri, kun påstanden.
|
|
||||||
// Henter nå brukerens faktiske GPS-posisjon FØR kartet initialiseres
|
|
||||||
// (kun ETT kall, samme som "Min posisjon nå"-veien -- ingen løpende
|
|
||||||
// watchPosition), med Oslo som fallback KUN hvis posisjon ikke kan
|
|
||||||
// hentes (avslått tillatelse, tidsavbrudd, ingen støtte).
|
|
||||||
function initMap(center: [number, number]) {
|
|
||||||
if (cancelled || !container) return
|
|
||||||
mapboxgl.accessToken = token as string
|
|
||||||
const map = new mapboxgl.Map({
|
|
||||||
container,
|
|
||||||
style: "mapbox://styles/mapbox/satellite-v9",
|
|
||||||
center,
|
|
||||||
zoom: 16,
|
|
||||||
attributionControl: false,
|
|
||||||
})
|
|
||||||
mapRef.current = map
|
|
||||||
|
|
||||||
map.on("load", () => setLoaded(true))
|
// Tap to place/move a single draggable marker — never re-inits the map.
|
||||||
map.on("error", () => setError("Kunne ikke laste kartet."))
|
map.on("click", (e) => {
|
||||||
|
const lngLat = { lng: e.lngLat.lng, lat: e.lngLat.lat }
|
||||||
// Tap to place/move a single draggable marker — never re-inits the map.
|
if (!markerRef.current) {
|
||||||
map.on("click", (e) => {
|
const marker = new mapboxgl.Marker({ draggable: true, color: "#d2551a" })
|
||||||
const lngLat = { lng: e.lngLat.lng, lat: e.lngLat.lat }
|
.setLngLat(e.lngLat)
|
||||||
if (!markerRef.current) {
|
.addTo(map)
|
||||||
const marker = new mapboxgl.Marker({ draggable: true, color: "#d2551a" })
|
marker.on("dragend", () => {
|
||||||
.setLngLat(e.lngLat)
|
const p = marker.getLngLat()
|
||||||
.addTo(map)
|
setPoint({ lng: p.lng, lat: p.lat })
|
||||||
marker.on("dragend", () => {
|
})
|
||||||
const p = marker.getLngLat()
|
markerRef.current = marker
|
||||||
setPoint({ lng: p.lng, lat: p.lat })
|
} else {
|
||||||
})
|
markerRef.current.setLngLat(e.lngLat)
|
||||||
markerRef.current = marker
|
}
|
||||||
} else {
|
setPoint(lngLat)
|
||||||
markerRef.current.setLngLat(e.lngLat)
|
})
|
||||||
}
|
|
||||||
setPoint(lngLat)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
const OSLO_FALLBACK: [number, number] = [10.7522, 59.9139]
|
|
||||||
if (typeof navigator !== "undefined" && navigator.geolocation) {
|
|
||||||
navigator.geolocation.getCurrentPosition(
|
|
||||||
(pos) => initMap([pos.coords.longitude, pos.coords.latitude]),
|
|
||||||
() => initMap(OSLO_FALLBACK),
|
|
||||||
{ enableHighAccuracy: true, timeout: 8000 },
|
|
||||||
)
|
|
||||||
} else {
|
|
||||||
initMap(OSLO_FALLBACK)
|
|
||||||
}
|
|
||||||
|
|
||||||
return () => {
|
return () => {
|
||||||
cancelled = true
|
|
||||||
markerRef.current?.remove()
|
markerRef.current?.remove()
|
||||||
markerRef.current = null
|
markerRef.current = null
|
||||||
mapRef.current?.remove()
|
map.remove()
|
||||||
mapRef.current = null
|
mapRef.current = null
|
||||||
}
|
}
|
||||||
}, [])
|
}, [])
|
||||||
|
|
|
||||||
File diff suppressed because one or more lines are too long
Loading…
Reference in a new issue