Skip to content

Conversation

@kenn
Copy link
Contributor

@kenn kenn commented Oct 19, 2025

Continue on #1051

This is totally optional, but I’ve also reorganized things further to show how simple the folder structure can be.


To remove the _ suffix completely, we could also do the following, but I left them unchanged.

password_.create.tsx
password.tsx



password.create.tsx
password.index.tsx

or

password/
  create.tsx
  index.tsx
$noteId_.edit.tsx
$noteId.tsx



$noteId.edit.tsx
$noteId.index.tsx

or

$noteId/
  edit.tsx
  index.tsx

Test Plan

All tests passed

Checklist

  • Tests updated
  • Docs updated

Screenshots

app/routes
├── $.tsx
├── me.tsx
├── _auth
│   ├── forgot-password.tsx
│   ├── login.tsx
│   ├── logout.tsx
│   ├── reset-password.tsx
│   ├── signup.tsx
│   ├── verify.tsx
│   ├── auth.$provider
│   │   ├── callback.ts
│   │   └── index.ts
│   ├── onboarding
│   │   ├── $provider.tsx
│   │   └── index.tsx
│   └── webauthn
│       ├── authentication.ts
│       └── registration.ts
├── _marketing
│   ├── about.tsx
│   ├── index.tsx
│   ├── privacy.tsx
│   ├── support.tsx
│   ├── tos.tsx
│   └── +logos
│       ├── logos.ts
│       └── ...
├── _seo
│   ├── robots[.]txt.ts
│   └── sitemap[.]xml.ts
├── admin
│   └── cache
│       ├── index.tsx
│       ├── lru.$cacheKey.ts
│       ├── sqlite.$cacheKey.ts
│       └── sqlite.tsx
├── resources
│   ├── download-user-data.tsx
│   ├── healthcheck.tsx
│   ├── images.tsx
│   └── theme-switch.tsx
├── settings
│   └── profile
│       ├── _layout.tsx
│       ├── change-email.tsx
│       ├── connections.tsx
│       ├── index.tsx
│       ├── passkeys.tsx
│       ├── password.tsx
│       ├── password_.create.tsx
│       ├── photo.tsx
│       └── two-factor
│           ├── _layout.tsx
│           ├── disable.tsx
│           ├── index.tsx
│           └── verify.tsx
└── users
    ├── index.tsx
    └── $username
        ├── index.tsx
        └── notes
            ├── $noteId.tsx
            ├── $noteId_.edit.tsx
            ├── _layout.tsx
            ├── index.tsx
            └── new.tsx

Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

This looks great. I don't want to swap the _ suffix in favor of index routes because it does technically change the routing structure and I think what we have now is more correct.

Thanks!

@kentcdodds kentcdodds merged commit 5b62cb9 into epicweb-dev:main Oct 23, 2025
6 checks passed
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