Skip to content

Move routes/index.ts to routes.ts #585

Description

@slifty

We define entity routes in our controllers which means that routes/index.ts being an index in a directory is not quite necessary / conveys an intention to have routes all defined in that directory.

FWIW I think routes in a routes directory is a reasonable approach but also think that stela's approach is legitimate (and better in some ways since it allows the dev to define auth middleware in line with route handler implementation)!

I just suggest we tweak this aspect of the implementation to be more clear of intent in directory structure.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions