search

RainEggplant / chatgpt-telegram-bot

A ChatGPT bot for Telegram based on Node.js. Support both browserless and browser-base APIs.
MIT License
323 stars 97 forks source link

problem with docker #26

Closed wangliangliang2 closed 1 year ago

wangliangliang2 commented 1 year ago

since .env is useless , code should be deleted. https://github.com/RainEggplant/chatgpt-telegram-bot/blob/4b4ecd51896e16155e6fc78c43bab4feac21188b/docker-compose.yml#L6

new version uses local.json, please insert code in Dockerfile

COPY config /app/config

https://github.com/RainEggplant/chatgpt-telegram-bot/blob/4b4ecd51896e16155e6fc78c43bab4feac21188b/Dockerfile#L26 or add code in docker-compose.yml 

volumes:
            - "$PWD/config:/app/config"

by the way , It's better to add code in docker-compose.yml

VectorZhao commented 1 year ago

I also have a need for this. After Github updates, it is a bit inconvenient to re-build Docker Image. I need to delete the GitHub project directory on the VPS first, and then replace the default.json, which is a bit cumbersome.

RainEggplant commented 1 year ago

OK, I'll refactor the docker-related files (like separating browserless and browser-based versions) when I have time. Already pushed a temporary fix in 668820e.

wangliangliang2 commented 1 year ago

OK, I'll refactor the docker-related files (like separating browserless and browser-based versions) when I have time. Already pushed a temporary fix in 668820e.

https://github.com/RainEggplant/chatgpt-telegram-bot/blob/668820e9b74d8e76b19c226a8ce04565e3051100/docker-compose.yml#L7

this temporary fix will not work. base on code, it need config not configs.

RainEggplant commented 1 year ago

@wangliangliang2 My bad. Fixed in 36593fb.