Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Configurable Server Middleware Path #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aturingmachine
Copy link

@aturingmachine aturingmachine commented Aug 26, 2021

Very much open to feedback on this as I am new to the codebase.

- adds middlewarePath default as '/_loading'
- uses options.middlewarePath for the path of the middleware that is
  pushed onto the nuxt serverMiddleware stack
@far-blue
Copy link

I'm late to this party as I'm just trying to get the _loading stuff working with a nuxt app that has multiple "base" paths so can't use the router base like usual.

One question - any reason for adding the new 'middlewarePath' option rather than using baseURLAlt? baseURLAlt appears to have been added to control the client-side URI but currently doesn't change the middleware URI so it makes sense to use the same in both places but, as I said, I'm new to this so I've possibly missed something obvious 😄

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