Skip to content

Conversation

@vivianyentran
Copy link
Contributor

  • Remove redirect instructions for next.config.js
  • Add redirect instructions for middleware.ts
  • Add redirect instructions for vercel.json

@vercel
Copy link

vercel bot commented Apr 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 26, 2024 8:32pm

@codecov
Copy link

codecov bot commented Apr 22, 2024

Bundle Report

Bundle size has no change ✅

## Add a New Redirect

Redirects are configured in [`next.config.js`](https://github.com/getsentry/sentry-docs/blob/master/next.config.js) and are checked before the filesystem (including pages and `/public` files ). The `redirects` function returns an array holding redirect objects.
There are two ways to add redirects in the Sentry docs:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stephanie-anderson @a-hariti Do either of you know middleware or vercel redirects take precendence? Is this something we want to call out?

Copy link
Contributor

@lizokm lizokm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Co-authored-by: Liza Mock <liza.mock@sentry.io>
@vivianyentran vivianyentran enabled auto-merge (squash) April 26, 2024 20:18
@vivianyentran vivianyentran merged commit 006d0b2 into master Apr 26, 2024
@vivianyentran vivianyentran deleted the update/contributing-redirects branch April 26, 2024 20:32
@github-actions github-actions bot locked and limited conversation to collaborators May 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants