Skip to content

fix(nitro): default to nitro generator and use resolved nitro routes#507

Merged
danielroe merged 1 commit intomainfrom
fix/override-routes
Sep 1, 2022
Merged

fix(nitro): default to nitro generator and use resolved nitro routes#507
danielroe merged 1 commit intomainfrom
fix/override-routes

Conversation

@danielroe
Copy link
Copy Markdown
Member

🔗 Linked issue

resolves #490

❓ Type of change

  • 📖 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

Although marking nitroGenerator as true by default, we were not actually checking the resolved options. This PR also updates the route generation to be based off nitro's prerender options, not the nuxt ones.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@danielroe danielroe added the bug Something isn't working label Aug 31, 2022
@danielroe danielroe self-assigned this Aug 31, 2022
@danielroe danielroe merged commit 6dfc006 into main Sep 1, 2022
@danielroe danielroe deleted the fix/override-routes branch September 1, 2022 08:07
@danielroe danielroe mentioned this pull request Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cant add prerender routes to nitro

1 participant