Repository navigation
Conversation
…nes empty state The message named the variable without saying it belongs to the ContextForge server, sending people to look for it in the web UI environment where it does not exist. Closes #7313 Signed-off-by: Anna Effort <anna.effort@ibm.com>
a-effort
requested review from
gandhipratik203,
gcgoncalves,
marekdano and
vishu-bh
October 22, 2026 02:36
There was a problem hiding this comment.
🟢 Approval recommended
The three localized wording updates address the stated ambiguity without changing application behavior or introducing identified issues.
No open issues 🎉
What changed in this PR
Clarifies the Home traffic card’s empty-state guidance: OBSERVABILITY_ENABLED=true must be set on the ContextForge server, not in the web UI.
Changes:
- Updates the English, Spanish, and Brazilian Portuguese messages to identify the server and instruct users to restart it.
- Preserves translation keys and application logic.
| File | Description |
|---|---|
| src/i18n/locales/pt-BR/dashboard.json | Clarifies server configuration and restart guidance in Brazilian Portuguese. |
| src/i18n/locales/es-ES/dashboard.json | Clarifies server configuration and restart guidance in Spanish. |
| src/i18n/locales/en-US/dashboard.json | Clarifies server configuration and restart guidance in English. |
Review effort: Balanced
This was referenced Oct 22, 2026
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
The empty state on the Home traffic card named
OBSERVABILITY_ENABLEDwithout saying where it goes. The setting belongs to the ContextForge server, not the web UI, and it appears nowhere in this repo outside the message, so anyone who looks for it here will not find it.Proposed change:
en-US, es-ES and pt-BR. No code change.
SystemSparklinesCard.test.tsxpasses unchanged, since it matches on "No traced requests".Checked against a local stack with
OBSERVABILITY_ENABLED=false.The es-ES and pt-BR wording is mine and would benefit from a native read. Both attach the restart as an enclitic pronoun on ContextForge.
Fixes IBM/mcp-context-forge#7313. This is interim: IBM/mcp-context-forge#7312 replaces the key with separate messages for observability being off and for no traffic, but it is blocked on IBM/mcp-context-forge#7311.