Skip to content

Fix broken Slack links in docs and skip slack.com in link checker - #5385

Merged
kathap merged 2 commits into
cloudfoundry:mainfrom
sap-contributions:fix-broken-slack-links-in-docs-v2
Aug 20, 2026
Merged

Fix broken Slack links in docs and skip slack.com in link checker#5385
kathap merged 2 commits into
cloudfoundry:mainfrom
sap-contributions:fix-broken-slack-links-in-docs-v2

Conversation

@WeiQuan0605

@WeiQuan0605 WeiQuan0605 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor
  • A short explanation of the proposed change:
  1. Update Slack URLs in docs from deprecated /messages/ format to /archives/ format

  2. Add slack.com to linksToSkip in gulpfile.js to skip Slack links in the link checker

  • An explanation of the use cases your change solves
    Slack returns 403 to automated link checkers due to bot detection, causing the docs CI job (checkdocs) to fail. The existing Slack URL in _introduction.md also used the deprecated /messages/capi/ format which no longer resolves correctly. This fix ensures the docs CI passes reliably without depending on Slack's availability for bots.

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@kathap kathap 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.

LGTM

@kathap
kathap merged commit 3bbba90 into cloudfoundry:main Aug 20, 2026
12 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Aug 20, 2026
Changes in cloud_controller_ng:

- Fix broken Slack links in docs and skip slack.com in link checker
    PR: cloudfoundry/cloud_controller_ng#5385
    Author: Wei Quan <64415962+WeiQuan0605@users.noreply.github.com>
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