Skip to content

fix: restore broken links on synchronizer traffic page - #1829

Merged
danielporterda merged 2 commits into
canton-network:mainfrom
bitdynamics-ab:fix/synchronizer-traffic-links
Oct 7, 2026
Merged

danielporterda merged 2 commits into
canton-network:mainfrom
bitdynamics-ab:fix/synchronizer-traffic-links

Conversation

@zheli

@zheli zheli commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Restores Sphinx :ref: / :term: links lost in the RST→MDX migration of the synchronizer traffic page (fixes Links to daml code is not rendered correctly #1828), including MemberTraffic / SynchronizerFeesConfig Daml API anchors.
  • Relinks in-page sections and glossary terms, and points related docs (SV ops, validator top-up, abort threshold, Grafana) at the correct section anchors.
  • Edits the networkvars template and regenerates tabs; updates test_splice_copied_cross_references expectations.

Test plan

  • Confirm MemberTraffic “(Daml API docs)” is a live link on /global-synchronizer/deployment/synchronizer-traffic
  • Confirm SynchronizerFeesConfig “Daml API docs” link works in each network tab
  • Spot-check in-page anchors (traffic accounting, parameters, top-ups, wasted traffic)
  • npm run validate:network-variable-tabs
  • pytest tests/test_splice_copied_cross_references.py

Sphinx :ref:/:term: targets were stripped during the RST→MDX migration,
leaving Daml API and in-page links as plain text. Restore markdown links
and tighten section anchors; update the copied-content cross-ref checks.

Fixes canton-network#1828

Signed-off-by: Zhe Li <linuxcity.jn@gmail.com>
@zheli
zheli force-pushed the fix/synchronizer-traffic-links branch from fed9590 to 3e0c37a Compare October 7, 2026 17:43
@danielporterda
danielporterda enabled auto-merge (squash) October 7, 2026 22:44

@danielporterda danielporterda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg2m. thanks for the issue report + rapid fix.

@danielporterda
danielporterda merged commit 4b2b0d1 into canton-network:main Oct 7, 2026
3 checks passed
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.

Links to daml code is not rendered correctly

2 participants