chatgpt_telegram_bot
chatgpt_telegram_bot copied to clipboard
[Feature] Add sqlite supports
In #45, we talk about add the sqlite as an alternative database for some budget machine, here's a quick implementation, detailed changes:
- Add sqlite database for local use only, usage:
python bot.py -d sqlite - Add http proxy supports:
python bot.py -p ip:port
Tested localy on my machine, everything work as expected
@karfly could you review and merge this PR, cause i don't have the write access.
Have you looked into deploying your own fork ? I am debating the pros and the cons about the usefulness of merging this to the main repo :thinking:
Have you looked into deploying your own fork ? I am debating the pros and the cons about the usefulness of merging this to the main repo 🤔
Emm, tbh, I think it is better to merge it into the upstream since the sqlite is less space consuming for personal usage. But also, I can keep it in my own fork if you guys doubt it's usesfulness.
As a user I'm waiting for this, if this doesn't get merged then some logic to clean old conversation data is really needed IMHO.
As for me better to have alternative database engines than create a lot of forks…
overall lgtm, just need to be tested and waiting for an approval
overall lgtm, just need to be tested and waiting for an approval
Should I initiate the test procedure or just test localy on my machine.
@clemsau could you tell me about how to initiate the test procedure, thks
Looking forward for it. MongoDB is a lot for personal and family usage.