Skip to content

docs(suites): fix broken links in docs#3399

Merged
kamilmysliwiec merged 1 commit intonestjs:masterfrom
MacMacky:fix/docs-recipe-suites-links
Apr 4, 2026
Merged

docs(suites): fix broken links in docs#3399
kamilmysliwiec merged 1 commit intonestjs:masterfrom
MacMacky:fix/docs-recipe-suites-links

Conversation

@MacMacky
Copy link
Copy Markdown
Contributor

@MacMacky MacMacky commented Apr 3, 2026

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Docs
  • Other... Please describe:

What is the current behavior?

The testing documentation links in the Suites Recipe didn't work or redirect to anything

Issue Number: N/A

What is the new behavior?

it fixes the links and it redirects to the testing documentation

Screen.Recording.2026-04-04.083503.mp4

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@kamilmysliwiec
Copy link
Copy Markdown
Member

That's odd, /fundamentals/testing is supposed to work fine. Instead of checking those links to absolute urls, we should investigate why md compiler doesn't correctly convert them to internal refs anymore

@MacMacky MacMacky closed this Apr 3, 2026
@MacMacky MacMacky reopened this Apr 4, 2026
@MacMacky MacMacky changed the title docs(suites): update testing documentation links docs(suites): fix broken links in docs Apr 4, 2026
@MacMacky MacMacky closed this Apr 4, 2026
@MacMacky MacMacky reopened this Apr 4, 2026
@MacMacky MacMacky force-pushed the fix/docs-recipe-suites-links branch from 67eb4b1 to 29225b8 Compare April 4, 2026 00:55
@MacMacky
Copy link
Copy Markdown
Contributor Author

MacMacky commented Apr 4, 2026

@kamilmysliwiec i've updated the PR for the fix, please review when you get a chance 🙏🏽 , thank you

@kamilmysliwiec kamilmysliwiec merged commit 77f3cfc into nestjs:master Apr 4, 2026
5 checks passed
@kamilmysliwiec
Copy link
Copy Markdown
Member

LGTM

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