Skip to content

Link fixes mostly related to ZNG spec update#1457

Merged
philrz merged 2 commits intomasterfrom
zng-spec-link-fixes
Oct 9, 2020
Merged

Link fixes mostly related to ZNG spec update#1457
philrz merged 2 commits intomasterfrom
zng-spec-link-fixes

Conversation

@philrz
Copy link
Contributor

@philrz philrz commented Oct 9, 2020

While it's convenient to be able to link directly to anchors for specific sections in docs as we've been doing, one of the hazards is that when a destination changes, the link doesn't show up as "broken" by our automation because the user just ends up being redirected to the top of the page in question. #1394 renumbered some sections in the ZNG spec, so here I've manually rechecked relevant anchor links and made sure they go to the right places again.

While I was at it, I found some absolute URLs that I changed to be relative. This makes for better links when working in branches.

@philrz philrz requested a review from a team October 9, 2020 18:27
@philrz philrz self-assigned this Oct 9, 2020
@philrz philrz merged commit 9df3fbe into master Oct 9, 2020
@philrz philrz deleted the zng-spec-link-fixes branch October 9, 2020 19:53
brim-bot pushed a commit to brimdata/zui that referenced this pull request Oct 9, 2020
…hilrz

This is an auto-generated commit with a zq dependency update. The zq PR
brimdata/super#1457, authored by @philrz,
has been merged.

Link fixes mostly related to ZNG spec update

While it's convenient to be able to link directly to anchors for specific sections in docs as we've been doing, one of the hazards is that when a destination changes, the link doesn't show up as "broken" by our automation because the user just ends up being redirected to the top of the page in question. brimdata/super#1394 renumbered some sections in the ZNG spec, so here I've manually rechecked relevant anchor links and made sure they go to the right places again.

While I was at it, I found some absolute URLs that I changed to be relative. This makes for better links when working in branches.
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