Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.
/frameworkPublic archive

docs: fix mdn links for 301 and 302 http status - #5416

Merged
danielroe merged 1 commit into
nuxt:mainfrom
MaximKing1:patch-1
Jun 11, 2022
Merged

docs: fix mdn links for 301 and 302 http status#5416
danielroe merged 1 commit into
nuxt:mainfrom
MaximKing1:patch-1

Conversation

@MaximKing1

Copy link
Copy Markdown
Contributor

On the https://v3.nuxtjs.org/guide/directory-structure/middleware page the links for 301 and 302 are broken.

❓ Type of change

The link was broken in the documentation

  • [* ] 📖 Documentation (updates to the documentation or readme)
  • [ *] 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The page https://v3.nuxtjs.org/guide/directory-structure/middleware shows the 302 and 301 links broken as seen below on the photo.
282EFAC4-E962-4B16-880A-046756222681

@netlify

netlifyBot commented Jun 11, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs ready!

NameLink
🔨 Latest commit956c6ff
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/62a46b8aed7f740008a5c642
😎 Deploy Previewhttps://deploy-preview-5416--nuxt3-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@MaximKing1MaximKing1 changed the title Fixed Link for 301 and 302 Links.fix(docs): Fixed Link for 301 and 302 Links.Jun 11, 2022
@MaximKing1MaximKing1 changed the title fix(docs): Fixed Link for 301 and 302 Links.docs(middleware): Fixed Link for 301 and 302 Links.Jun 11, 2022
@danielroedanielroe changed the title docs(middleware): Fixed Link for 301 and 302 Links.docs: fix mdn links for 301 and 302 http statusJun 11, 2022
@danielroe
danielroe merged commit 75db904 into nuxt:mainJun 11, 2022
@pi0pi0 mentioned this pull request Jun 12, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@MaximKing1@danielroe