search

agnyz / the-bed-stack

Bun + ElysiaJS + DrizzleORM = the stack you don't want to sleep on 🛌💤
https://agnyz.github.io/the-bed-stack/
MIT License
39 stars 3 forks source link

Rename `auth.ts` #52

Closed yamcodes closed 1 year ago

yamcodes commented 1 year ago

We should rename the src/auth.ts file to clarify its scope as a clear component in our architecture.

In #47, I offered to rename it to auth.middleware.ts or auth.service.ts, but @Hajbo brought up the fact it currently is out of scope of a service or middleware.

We should either make it one of those, or figure out if utils are in the best practices of Nest.js.

Hajbo commented 1 year ago

We can probably make it into a service if we really want to

yamcodes commented 1 year ago

We can probably make it into a service if we really want to

Yes! I really want to.

It would make all my dreams come true.

Please check the linked PR.