This repository was archived by the owner on Jul 5, 2026. It is now read-only.
fix(footer): one status dot, real freshness line, endpoint in the bottom bar - #269
Merged
Conversation
…tom bar Footer polish per review: - Drop the redundant 'healthy' word from the endpoint pill — the green dot already says it. Visible text is now just latency (or 'unreachable' / 'checking…'); the word form stays in the tooltip/aria for a11y. - Collapse three pulsing green dots down to one. Removed the 'registry pulse' mg-live-dot and the 'live registry' pulse dot; the only live dot left is the endpoint-health one (which actually conveys status). - Move the API endpoint to the bottom bar (where 'live registry' was) and drop 'live registry'. - Replace the vanity 'Registry Pulse' label with a real 'updated <relative>' freshness indicator from the freshness feed's generated_at. - Replace the vague '— unofficial' with the project's own accurate disclaimer, 'Not an OpenTensor/Bittensor product'.
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
metagraphed-ui | 7432f66 | Commit Preview URL Branch Preview URL |
Jun 26 2026, 11:41 AM |
Run both brand links through safeExternalUrl with a known-good fallback, the same guard the API links use, so a misconfigured VITE_METAGRAPHED_REPO/_DISCORD override can't inject an unsafe href. #268 only covered Discord and was cut against the pre-rework footer; this covers GitHub too, on the current footer, computed once at module scope.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up polish on the footer (PR #267), addressing review feedback:
api.metagraph.sh/api/v1 · 142 ms), orunreachable/checking…. The word form stays in the tooltip + aria for colour-blind / screen-reader users.mg-live-dotand the "live registry" pulse dot; the only live dot left is the endpoint-health one (the one that actually means something).updated <relative>freshness indicator sourced from the freshness feed'sgenerated_at(guarded to a string — the type carries an index signature).Net diff is
app-shell.tsxonly (theuseEndpointHealthhook +--health-badtoken landed in #267).Verification:
prettier --checkclean ·tsc --noEmitclean ·eslint0 errors · 160/160 tests ·vite buildsucceeds.