kiliman / remix-flat-routes

Remix package to define routes using the flat-routes convention
MIT License
640 stars 22 forks source link

Unclear docs about integration with Remix Core #92

Closed cjoecker closed 7 months ago

cjoecker commented 7 months ago

In the docs, It says the remix-flat-routes will be integrated to remix core.

For me, the following is not clear:

  1. What features of remix-flat-routes will be integrated in Remix Core
  2. What features will stay only in remix-flat-routes in the near feature
  3. When will remix-flat-routes be integrated in Remix Core (if it is not already integrated)
kiliman commented 7 months ago

I suppose I need to update that part of the README.

remix-flat-routes was the initial implementation of the flat routes spec. I added some enhancements to it based on user feedback.

When Ryan decided to make flat routes the default convention, he opted to only use the original spec. This is what is now the v2 routing convention.

So if you want hybrid routes, extended route filenames, etc., you'll still need to use the remix-flat-routes package. This package will always maintain compatibility with the Remix convention. It is simply a superset/extension.

kiliman commented 7 months ago

Also note that popular packages like Kent's Epic Stack uses remix-flat-routes, so although not a core, it is stable.

https://github.com/epicweb-dev/epic-stack/blob/802fd4a83b979c17a01408241fb721ec428bf034/package.json#L152

cjoecker commented 7 months ago

@kiliman thanks for the fast answer! An update of the docs would be helpful, thanks!

kiliman commented 7 months ago

Done!