search

netlify / remix-template

Deploy your Remix site to Netlify
83 stars 27 forks source link

Move to Remix routing v2 #106

Closed nickytonline closed 1 year ago

nickytonline commented 1 year ago

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

The current routing logic is deprecated. When you run remix build, the following deprecation warning appears.

⚠️ DEPRECATED: The old nested folders route convention has been deprecated in favor of "flat routes".  Please enable the new routing convention via the `future.v2_routeConvention` flag in your `remix.config.js` file.  For more information, please see https://remix.run/docs/en/main/file-conventions/route-files-v2.

Describe the solution you'd like A clear and concise description of what you want to happen.

Upgrade to the v2 routing as pre the instructions in https://remix.run/docs/en/main/file-conventions/route-files-v2

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

nickytonline commented 1 year ago

The new routing is already available so this PR is no longer necessary. Also, I updated the routing naming for the index route in https://github.com/netlify/remix-template/pull/111.