{"id":"69afdf6cf8f0bf680691c49b","title":"2026 03 10","path":"how-to/committee/release-notes/2026-03-10","contentMarkdown":"# 10-Mar-2026 — map hang, GPX route parsing, mail API resilience, walk search layout\n\n## [build 515](https://github.com/nbarrett/ngx-ramblers/actions/runs/22894482875) — [commit 0cad242](https://github.com/nbarrett/ngx-ramblers/commit/0cad242aac84cb3cef60ff906d86c1a8304cddd0)\n\n_____\n\n## GPX Parser: Support Route Elements\n- **Root cause**: GPX parser only handled `/` (track)\nelements but ignored `/` (route) elements. 37 of 39\nSevenoaks walking route GPX files use `` format, so only 2\nroutes were rendering on the map\n- **Fix**: added `parseRoute()` and `parseRoutePoints()` methods to\nconvert `/` elements into renderable `GpxTrack` objects\nalongside existing `` parsing\n- **Waypoint dedup**: changed `parseWaypoints()` from selecting\n`\"wpt, rtept\"` to just `\"wpt\"` — `` elements are now parsed\nas route tracks, not waypoints, preventing duplicate rendering\n## Map Viewport Hang Fix\n- **Root cause**: Dragging/zooming the map triggered an infinite feedback\nloop — `moveend` → `loadRoutes()` → layer reassignment + `fitBounds`\n→ Leaflet re-render → `moveend` again\n- **Debounced viewport handler**: 300ms debounce on `moveend`/`zoomend`\nprevents rapid-fire reloads during panning\n- **Re-entrancy guard**: `loadRoutesInProgress` flag skips concurrent\n`loadRoutes()` calls\n- **Suppress programmatic events**: `suppressViewportHandler` flag\nprevents layer/bounds assignment from re-triggering the viewport\nhandler (cleared after 200ms)\n- **Skip fitBounds on viewport reloads**: `loadRoutes(skipFitBounds)`\nparameter prevents `calculateFitBounds` on user-initiated pan/zoom,\nwhich was the primary loop trigger\n- **Removed cache eviction on viewport change**: the viewport filtering\nalready uses `currentBounds.intersects()` on cached data, so deleting\nand re-fetching GPX on every drag was unnecessary\n## Mail API Resilience (No API Key / Invalid Key)\n- **Short-circuit when no API key**: `MailMessagingService.initialise()`\nchecks `mailConfig.apiKey` after loading config — if absent, skips all\nBrevo API calls (account, lists, folders, templates) and sets safe\nempty defaults. Zero 401 errors in the console\n- **Short-circuit on account error**: if the account query fails (invalid\nkey), remaining Brevo calls are skipped with empty defaults\n- **Senders list guard**: `MailSendersListComponent` waits for the\n`mailMessagingConfig` via the `events()` observable before calling\nsenders/domain APIs — skips if `accountError` is set\n- **Create-or-amend sender guard**: checks `brevoAccountConfigured()`\nbefore calling `querySenders()`\n- **Mail provider stats guard**: `calculateMailProviderStats()` handles\n`this.list` being undefined when Brevo lists fail to load, showing\n\"No Brevo lists available\" instead of crashing\n- **List key values null safety**: added `?.` and `|| []` fallback when\n`brevo.lists.lists` is undefined\n- **Alert icon**: changed error alert icon from `faCircleCheck` to\n`faExclamationTriangle` for the account error panel\n## CommonDataService Double HTTP Request Fix\n- **Root cause**: `responseFrom()` used `.subscribe()` + `.toPromise()`\non a shared observable, causing two subscriptions and duplicate HTTP\nrequests\n- **Fix**: replaced with single `firstValueFrom()` call — one\nsubscription, one HTTP request, proper error propagation via `throw`\n## Config Save: Allow Clearing Sensitive Fields\n- **Root cause**: `restoreSensitiveFields()` in the backend config\ncontroller treated both missing keys AND falsy values (`\"\"`, `null`)\nas \"restore from DB\", making it impossible to clear the API key\n- **Fix**: changed condition from `!(k in incoming) || !incoming[k]` to\n`!(k in incoming)` — only restores when the key is truly absent\n(redacted), not when explicitly cleared\n## Mail Settings: Prevent Config Overwrite During Editing\n- **Root cause**: `MailMessagingService.initialise()` created new\nsubscriptions to `committeeConfig` and `systemConfigService` on every\ncall, causing accumulated subscriptions that re-emitted config and\noverwrote unsaved edits\n- **Fix**: moved one-time subscriptions to the constructor;\n`acceptNextConfigEmission` flag in `MailSettingsComponent` prevents\nsubscription emissions from overwriting local edits except after\nsave or undo\n## Domain API URL Refactor\n- Changed domain endpoints from path parameters\n(`/domains/:domainName/configuration`) to query parameters\n(`/domains/configuration?domainName=...`) across backend routes and\nfrontend service\n## Walk Search Layout\n- Alert inline when default filtering, own row with pagination/filters\n- Clear filters resets to default preset\n- Bug button hidden by default\n## Events\n- Bug button visibility fix","contentHtml":"

10-Mar-2026 — map hang, GPX route parsing, mail API resilience, walk search layout

\n

build 515 — commit 0cad242

\n
\n

GPX Parser: Support Route Elements

\n
    \n
  • Root cause: GPX parser only handled <trk>/<trkpt> (track)\nelements but ignored <rte>/<rtept> (route) elements. 37 of 39\nSevenoaks walking route GPX files use <rte> format, so only 2\nroutes were rendering on the map
    • \n
  • Fix: added parseRoute() and parseRoutePoints() methods to\nconvert <rte>/<rtept> elements into renderable GpxTrack objects\nalongside existing <trk> parsing
    • \n
  • Waypoint dedup: changed parseWaypoints() from selecting\n"wpt, rtept" to just "wpt" — <rtept> elements are now parsed\nas route tracks, not waypoints, preventing duplicate rendering
    • \n
\n

Map Viewport Hang Fix

\n
    \n
  • Root cause: Dragging/zooming the map triggered an infinite feedback\nloop — moveend → loadRoutes() → layer reassignment + fitBounds\n→ Leaflet re-render → moveend again
    • \n
  • Debounced viewport handler: 300ms debounce on moveend/zoomend\nprevents rapid-fire reloads during panning
    • \n
  • Re-entrancy guard: loadRoutesInProgress flag skips concurrent\nloadRoutes() calls
    • \n
  • Suppress programmatic events: suppressViewportHandler flag\nprevents layer/bounds assignment from re-triggering the viewport\nhandler (cleared after 200ms)
    • \n
  • Skip fitBounds on viewport reloads: loadRoutes(skipFitBounds)\nparameter prevents calculateFitBounds on user-initiated pan/zoom,\nwhich was the primary loop trigger
    • \n
  • Removed cache eviction on viewport change: the viewport filtering\nalready uses currentBounds.intersects() on cached data, so deleting\nand re-fetching GPX on every drag was unnecessary
    • \n
\n

Mail API Resilience (No API Key / Invalid Key)

\n
    \n
  • Short-circuit when no API key: MailMessagingService.initialise()\nchecks mailConfig.apiKey after loading config — if absent, skips all\nBrevo API calls (account, lists, folders, templates) and sets safe\nempty defaults. Zero 401 errors in the console
    • \n
  • Short-circuit on account error: if the account query fails (invalid\nkey), remaining Brevo calls are skipped with empty defaults
    • \n
  • Senders list guard: MailSendersListComponent waits for the\nmailMessagingConfig via the events() observable before calling\nsenders/domain APIs — skips if accountError is set
    • \n
  • Create-or-amend sender guard: checks brevoAccountConfigured()\nbefore calling querySenders()
    • \n
  • Mail provider stats guard: calculateMailProviderStats() handles\nthis.list being undefined when Brevo lists fail to load, showing\n"No Brevo lists available" instead of crashing
    • \n
  • List key values null safety: added ?. and || [] fallback when\nbrevo.lists.lists is undefined
    • \n
  • Alert icon: changed error alert icon from faCircleCheck to\nfaExclamationTriangle for the account error panel
    • \n
\n

CommonDataService Double HTTP Request Fix

\n
    \n
  • Root cause: responseFrom() used .subscribe() + .toPromise()\non a shared observable, causing two subscriptions and duplicate HTTP\nrequests
    • \n
  • Fix: replaced with single firstValueFrom() call — one\nsubscription, one HTTP request, proper error propagation via throw
    • \n
\n

Config Save: Allow Clearing Sensitive Fields

\n
    \n
  • Root cause: restoreSensitiveFields() in the backend config\ncontroller treated both missing keys AND falsy values ("", null)\nas "restore from DB", making it impossible to clear the API key
    • \n
  • Fix: changed condition from !(k in incoming) || !incoming[k] to\n!(k in incoming) — only restores when the key is truly absent\n(redacted), not when explicitly cleared
    • \n
\n

Mail Settings: Prevent Config Overwrite During Editing

\n
    \n
  • Root cause: MailMessagingService.initialise() created new\nsubscriptions to committeeConfig and systemConfigService on every\ncall, causing accumulated subscriptions that re-emitted config and\noverwrote unsaved edits
    • \n
  • Fix: moved one-time subscriptions to the constructor;\nacceptNextConfigEmission flag in MailSettingsComponent prevents\nsubscription emissions from overwriting local edits except after\nsave or undo
    • \n
\n

Domain API URL Refactor

\n
    \n
  • Changed domain endpoints from path parameters\n(/domains/:domainName/configuration) to query parameters\n(/domains/configuration?domainName=...) across backend routes and\nfrontend service
    • \n
\n

Walk Search Layout

\n
    \n
  • Alert inline when default filtering, own row with pagination/filters
    • \n
  • Clear filters resets to default preset
    • \n
  • Bug button hidden by default
    • \n
\n

Events

\n
    \n
  • Bug button visibility fix
    • \n
\n"}