-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Please support sqlite #367
Comments
Hi @jvoisin, Sqllite is not currently a dependency, even for search. I can see how sqlite may be better for smaller instances though. The new changes to the search system makes support for sqlite much more possible now. I just had a go (b30ee10) but it is going to take some additional care to implement, Don't really want to do this right before a release. Need to also consider the additional amount of work and support this will need going forward. It's close similarities to MySQL helps though. |
Damn, I misread the ticket about the possibility of using sqlite for plain-text search, sorry. Thank you for considering this ♥ |
Just in-case someone is interested in this feature, More in-depth conversions about non-MySQL support can be found in #76. |
What would be the first steps to developing SQLite support? |
@partoneoftwo In regards to the technical aspect, It'd be a case of adding in the sqllite part of the default laravel config into the similar BookStack file, updating the config to point at a sqllite db, then migrate and fix any issues from there. In regards to getting support into the project, It would require my faith in a longer term contributor. I've detailed my feelings for additional DB support here: My feelings remain much the same. |
Thanks for the thoughts. About the error messages: Sidenote: |
+1 |
May I ask if we are still considering supporting sqlite |
+1 |
Diversity in database back-ends is good. |
It would be great to support sqlite as database backend:
The text was updated successfully, but these errors were encountered: