Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ditch Sequelize #43

Open
KNOXDEV opened this issue Apr 27, 2022 · 1 comment
Open

Ditch Sequelize #43

KNOXDEV opened this issue Apr 27, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@KNOXDEV
Copy link
Member

KNOXDEV commented Apr 27, 2022

Long story short its basically just a piece of shit.

Overall it is definitely an improvement over what we had before, but it has caused a large number of issues. It has very weak support for aggregations and typescript, and there are many places in the code where I have to waste lots of time fighting it to get simple functionality.

Most of all it has singlehandedly made the entire codebase ugly.

I don't know what we should use instead, but I'm not above also just writing something better ourselves. There's another option called Prisma that might be worth looking into as well.

@KNOXDEV KNOXDEV added the enhancement New feature or request label Apr 27, 2022
@StormFireFox1
Copy link
Member

My own two cents from personal experience:

Prisma is very nice to demo databases with and it's quick for simple CRUD projects; TypeScript support is good, and there's built-in aggregations and grouping, etc.

The only mildly annoying part is that the query language is object-based with not a lot of documentation rather than just using functions. I think because of that there's a "Prisma DSL" you kinda have to learn. It's not bad, but it takes some getting used to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants