Skip to content

fix: dev server should redirect when a page doesn't exist#7841

Merged
ematipico merged 3 commits intomainfrom
fix/hanle-redirect-in-dev-plt-734
Jul 28, 2023
Merged

fix: dev server should redirect when a page doesn't exist#7841
ematipico merged 3 commits intomainfrom
fix/hanle-redirect-in-dev-plt-734

Conversation

@ematipico
Copy link
Copy Markdown
Member

Changes

Closes #7752

Testing

I tested locally using a project. I tried to create a test, but the testing suite is not suitable for this case, it seems.

Docs

N/A

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jul 27, 2023

🦋 Changeset detected

Latest commit: ebc69fc

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Jul 27, 2023
@matthewp
Copy link
Copy Markdown
Contributor

Missing a changeset. One nit comment, use your judgment on that one.

@ematipico ematipico requested a review from matthewp July 27, 2023 15:54
@ematipico
Copy link
Copy Markdown
Member Author

@matthewp I changed the logic. There were few regressions while running tests. Now it should be fine.

@ematipico ematipico force-pushed the fix/hanle-redirect-in-dev-plt-734 branch from 2227734 to ebc69fc Compare July 27, 2023 16:34
@ematipico ematipico changed the title fix: middleware should redirect when a page doesn't exist fix: dev server should redirect when a page doesn't exist Jul 27, 2023
@ematipico ematipico merged commit 2275c7d into main Jul 28, 2023
@ematipico ematipico deleted the fix/hanle-redirect-in-dev-plt-734 branch July 28, 2023 08:20
This was referenced Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Middleware redirect does not work if the page file doesn't exist

3 participants