Skip to content

fix edge middleware path on windows #839

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Apr 28, 2025

Conversation

james-elicx
Copy link
Contributor

@james-elicx james-elicx commented Apr 24, 2025

  • Normalizes the path that gets used for the middleware in the edge plugins.

Copy link

changeset-bot bot commented Apr 24, 2025

🦋 Changeset detected

Latest commit: 6dbd719

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

This PR includes changesets to release 3 packages
Name Type
@opennextjs/aws Patch
app-pages-router Patch
app-router Patch

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

Copy link

pkg-pr-new bot commented Apr 24, 2025

Open in StackBlitz

pnpm add https://pkg.pr.new/@opennextjs/aws@839

commit: 6dbd719

@james-elicx james-elicx changed the title fix windows paths fix edge middleware path on windows Apr 27, 2025
@james-elicx james-elicx marked this pull request as ready for review April 27, 2025 19:16
Copy link
Contributor

@vicb vicb left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Contributor

@conico974 conico974 left a comment

Choose a reason for hiding this comment

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

Thanks

@james-elicx
Copy link
Contributor Author

fyi, I can't merge PRs in this repo.

@vicb vicb merged commit de53c4d into opennextjs:main Apr 28, 2025
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants