Skip to content

Fix: show the API path hint when a bare address redirects - #331

Merged
SandObserver merged 1 commit into
mainfrom
fix/api-path-hint-on-redirect
Sep 24, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/api-path-hint-on-redirect

Conversation

@SandObserver

@SandObserver SandObserver commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Show the missing-API-path hint for a bare address that answers with a redirect (upstream.redirect), the usual result of a service root sending the request to its login page.
  • Stop treating a plain invalid code as a missing path: the badge proxy returns it only for malformed requests, and a page that is not data already reaches the no-values hint on the success path.

Testing
All green. Moved the redirect case into the hint test and invalid into the known-cause test.

Changelog

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.

@SandObserver SandObserver added bug Something isn't working ui/ux UI, UX, and accessibility only fix labels Sep 24, 2026
@SandObserver
SandObserver force-pushed the fix/api-path-hint-on-redirect branch from fff96e2 to eadfdaa Compare September 24, 2026 03:11
@SandObserver
SandObserver merged commit 41fc5d4 into main Sep 24, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/api-path-hint-on-redirect branch September 24, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ui/ux UI, UX, and accessibility only fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant