-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add support for views #29
Comments
@ad-si I'm not that familiar with views, so I guess that's the fundamental problem 😄 My 5 minute read seems to show that Sequelize (which tuql uses under the hood) doesn't support SQLite views, so that may be a blocker. |
Well, views are just queries which yield a new (read only) table. Reading from a table or a view works exactly in the same way, so it's really straight forward.
What the 😳. I guess that's why I never use ORMs. Completely useless abstraction. Since this project is explicitly for SQLite databases, maybe you shouldn't use it either? I feel like having direct access to all SQLite features would be handy… |
Not completely useless, it allowed for this library to be created quickly without writing a bunch of SQL plumbing myself 🤷♂ Regardless, views are not something I have needed for this, happy to look at a PR that brings support for them though. |
Is there a fundamental problem preventing this?
The text was updated successfully, but these errors were encountered: