Repository navigation
fix(docs): link the CNCF Slack invite from the Research User Group page - #2001
Merged
Merged
Conversation
The Research page was the only page under docs/community/user-groups/ to link a workspace URL (cloud-native.slack.com/archives/...). That URL redirects to the workspace root and answers 403 to anyone not already signed in, so the reader a 'How to participate' list exists for reaches a sign-in wall rather than a way to join. Its three sibling pages name the channel in text and link the open invite page. It also makes the weekly check-links job report a dead link on main, on a URL no open pull request introduced. The guard belongs with the other User Group page contracts, so a group added later cannot reintroduce the deep link unnoticed. Closes #2000 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: quality <280983584+hivecommons-hive[bot]@users.noreply.github.com>
Contributor
Author
|
Important Held for human review by the hive's ACMM level gate. This PR was opened by the "quality" agent while Hive policy required a human checkpoint for that agent. Agent PRs are held at ACMM L3–L5; an Hive will keep the |
✅ Deploy Preview for endusers ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: mrbobbytables <mrbobbytables@users.noreply.github.com>
mrbobbytables
added this pull request to the merge queue
Oct 30, 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.
Test Improvement
docs/community/user-groups/research/index.mdlinked#ug-researchashttps://cloud-native.slack.com/archives/CTW1NRLBW. That is a workspace-internal URL: it redirects to the workspace root and answers 403 to an anonymous client (verified three times withcurl). A reader already signed in to CNCF Slack reaches the channel; everyone else — the audience a "How to participate" list is written for — reaches a sign-in wall with no route to join.The three sibling pages (
public-sector,telecom,platform-engineering) already name the channel in text and linkhttps://slack.cncf.io/, the open invite page. Research, added in #1996 on 2026-10-29, was the only one that did not.It also made
npm run check:linksreport1 dead link found. The weeklycheck-linksworkflow last ran green on 2026-10-26, before that merge, so it would have gone red onmainat its next scheduled run.Changes
docs/community/user-groups/research/index.md— link the invite page, name the channel in text, matching the three sibling pages.tests/community-group-coverage.test.mjs— one test asserting no page underdocs/community/user-groups/links aslack.comworkspace URL. It joins the page/report/sidebar contracts already in that file, so the next group page cannot reintroduce the deep link unnoticed.No other file, function or page is touched.
Verification
node --test tests/community-group-coverage.test.mjs— 7/7 pass; reverting only thedocs/hunk fails the new test with the file name in the message.npm run test:unit— 3764 pass, 0 fail.npm run check:format,check:lint,check:spelling,check:duplication,check:unused— clean.npx markdown-link-check --config .markdown-link-check.json docs/community/user-groups/research/index.md— 6 links checked, 0 dead.Remit
ADR 0008 rule 1: a defect an end user can observe on the production site (a broken navigation path), and a required check that would fail on
mainfor a reason unrelated to any open pull request. The test is the guard the fix needs, not a coverage-metric change.Related Issue
Closes #2000
Filed by quality agent (hold-gated mode). Human review required.
— hive: agent=quality backend=copilot model=claude-opus-5 copilot=1.0.88