chatgpt_telegram_bot icon indicating copy to clipboard operation
chatgpt_telegram_bot copied to clipboard

[Feature] Add sqlite supports

Open chraac opened this issue 2 years ago • 9 comments

In #45, we talk about add the sqlite as an alternative database for some budget machine, here's a quick implementation, detailed changes:

  1. Add sqlite database for local use only, usage:
    python bot.py -d sqlite
    
  2. Add http proxy supports:
    python bot.py -p ip:port
    

Tested localy on my machine, everything work as expected

chraac avatar Feb 20 '23 14:02 chraac

@karfly could you review and merge this PR, cause i don't have the write access.

chraac avatar Feb 22 '23 08:02 chraac

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:

clemsau avatar Mar 16 '23 10:03 clemsau

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.

chraac avatar Mar 16 '23 10:03 chraac

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.

madiele avatar Mar 16 '23 12:03 madiele

As for me better to have alternative database engines than create a lot of forks…

nikbyte avatar Mar 16 '23 12:03 nikbyte

overall lgtm, just need to be tested and waiting for an approval

clemsau avatar Mar 16 '23 14:03 clemsau

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.

chraac avatar Mar 22 '23 11:03 chraac

@clemsau could you tell me about how to initiate the test procedure, thks

chraac avatar Apr 19 '23 02:04 chraac

Looking forward for it. MongoDB is a lot for personal and family usage.

PlugFox avatar Feb 14 '24 19:02 PlugFox