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

Add SQLite backend to Convos #615

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Add SQLite backend to Convos #615

wants to merge 3 commits into from

Conversation

jhthorsen
Copy link
Collaborator

This is currently just a draft to show how a new backend might look. I don't know when I will have time to finish this, so help is greatly appreciated.

TODO:

  • Add a database schema
  • Make sure objects and messages are inflated and deflated properly. (Some columns might contain JSON)
  • Duplicate the ::File tests, but with $ENV{CONVOS_BACKEND} = 'Convos::Core::Backend::SQLite'
  • Write a migration script, so existing ::File users can convert to ::SQLite without loosing data
  • Add a detection step to "convos install" which should default to SQLIte, if SQLite is available on the system

There's probably more...

@brunoramoslu
Copy link

Is this still in the roadmap as a possible/interesting feature to implement?
Could work be picked up, or continued, from here?

@jhthorsen
Copy link
Collaborator Author

It's not on the roadmap, since the file backend works pretty good for me. It would however open new possibilities (like better search capabilities) so it would be awesome if someone could finish it 👍

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

Successfully merging this pull request may close these issues.

None yet

2 participants