cleanup: simplify geocoding provider configuration

barrettruth commented

2026-05-11 19:19:50 +00:00

Owner

Parent: #305

Problem

Geocoding currently supports Photon plus token-backed providers through settings and test endpoints. That may be more provider surface than the blank-start app needs, and the provider rules are split across UI, route handlers, and integration config helpers.

Solution

Make the geocoding scope explicit and smaller:

decide whether v0.1 keeps Photon-only, Photon plus env-token provider, or full provider selection
centralize provider IDs, labels, required fields, and test behavior if multiple providers remain
remove settings/test UI for providers that are not part of v0.1
keep geocoding failures visible but non-blocking for task creation/editing

Acceptance criteria

The settings UI exposes only supported geocoding behavior.
/api/geocode and provider test routes use the same provider contract.
No paid-provider credential is required for a fresh blank-start instance.
Tests cover the selected provider fallback behavior.

Parent: #305 ## Problem Geocoding currently supports Photon plus token-backed providers through settings and test endpoints. That may be more provider surface than the blank-start app needs, and the provider rules are split across UI, route handlers, and integration config helpers. ## Solution Make the geocoding scope explicit and smaller: - decide whether v0.1 keeps Photon-only, Photon plus env-token provider, or full provider selection - centralize provider IDs, labels, required fields, and test behavior if multiple providers remain - remove settings/test UI for providers that are not part of v0.1 - keep geocoding failures visible but non-blocking for task creation/editing ## Acceptance criteria - The settings UI exposes only supported geocoding behavior. - `/api/geocode` and provider test routes use the same provider contract. - No paid-provider credential is required for a fresh blank-start instance. - Tests cover the selected provider fallback behavior.

barrettruth added this to the v0.1.0 milestone

2026-05-11 19:19:50 +00:00

barrettruth added the

track:api

track:ui

type:cleanup

labels

2026-05-11 19:19:50 +00:00

barrettruth referenced this issue

2026-05-11 19:20:27 +00:00

refactor: centralize provider registry for settings integrations #325

barrettruth referenced this issue

2026-05-11 19:20:49 +00:00

tracker: blank-start cleanup and foundation migration #305

barrettruth referenced this issue from a pull request that will close it,

2026-05-11 19:58:17 +00:00

Simplify geocoding provider configuration #338

barrettruth referenced this issue

2026-05-11 20:04:51 +00:00

Simplify geocoding provider configuration #338

barrettruth commented

2026-05-11 20:05:06 +00:00

Author

Owner

Product direction update: NLP and geocoding/mapping providers must stay. Do not solve this issue by making geocoding Photon-only or removing Mapbox/Google Maps support. The remaining #315 work should simplify/refactor provider configuration while preserving provider choices, credential handling, tests, and non-blocking fallback behavior.

barrettruth referenced this issue from a pull request that will close it,

2026-05-11 20:21:31 +00:00

cleanup: centralize geocoding provider config #350

~~barrettruth referenced this issue 2026-05-11 20:27:12 +00:00~~

refactor: split task panel controller #353

barrettruth closed this issue

2026-05-11 20:29:39 +00:00

barrettruth referenced this issue from a commit

2026-05-11 20:29:40 +00:00

cleanup: centralize geocoding provider config (#350)

barrettruth referenced this issue

2026-05-11 20:31:29 +00:00