Skip to content

fix: nav highlight, voted-action metadata errors and IP redaction for v3.0.0 - #4298

Merged
mesudip merged 4 commits into
developfrom
fix/nav-highlight-diagnostics-ip-redaction
Oct 18, 2026
Merged

mesudip merged 4 commits into
developfrom
fix/nav-highlight-diagnostics-ip-redaction

Conversation

@mesudip

@mesudip mesudip commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

List of changes

  • Fix: "Governance Actions" in the navigation is highlighted while Proposals, Live Voting or Historical is open
  • Fix: an action you have voted on shows its metadata error and "View Error Report" like any other action
  • Change: the backend hides the middle of IP addresses in metadata reports and error responses, e.g. 203.xx.xx.45 (D171)

Meant for v3.0.0: promote to test and main before the tag is cut.

Checklist

  • related issue: none
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works (nav and IP redaction; the voted-action fix has none)

The parent of the Proposals, Live Voting and Historical links now shows as active whenever findActiveNavTo matches one of its children, in the top bar, the dashboard sidebar and both mobile menus. Link honours its isActive override even with an empty navTo, and FakeLink and DrawerLink take one.
The details card hid the metadata error and its fetch report once the wallet had voted on the action or while its vote was in progress. The action's metadata fails the same either way, so the card no longer passes the vote state to DataMissingInfoBox.
The metadata routes and error bodies built from data-layer errors send IPv4 addresses as 203.xx.xx.45 and IPv6 addresses with only their first and last group. Document bodies, urls, ids and hashes are left as the service sent them. Amends D116.
@mesudip
mesudip force-pushed the fix/nav-highlight-diagnostics-ip-redaction branch from dc0a4e7 to 761e09c Compare October 18, 2026 18:18
@mesudip
mesudip changed the base branch from test to develop October 18, 2026 18:18
@mesudip
mesudip merged commit e6a22b6 into develop Oct 18, 2026
14 checks passed
@mesudip
mesudip deleted the fix/nav-highlight-diagnostics-ip-redaction branch October 18, 2026 18:45
This was referenced Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants