Closed max298 closed 1 year ago
I think we should break something deliberately before merging this. Perhaps change the namespace of the package?
maybe change the variable naming?
CHATGPT_MODEL=
maybe change the variable naming?
CHATGPT_MODEL=
Yeah that could do it. Something that requires user intervention.
I like that idea! I'll implement the change shortly.
I just noticed the env variable is optional and has a default, so a change of the variable will most likely not even be noticed by our users if they didn't change the model.
Lets make it a required field with no default and adjust the name.
I've also added an log entry if the new CHATGPT_API_MODEL
is unset resulting in the bot not starting.
I think this is now ready for review 😃
This is kind of a "dangerous" change, as the OpenAI accounts of our users will be charged by simply updating the version to this one.
I'm not sure if we should adjust the readme any further in order to make this clear, so everyone knows the usage is no longer free (and most likely will never become free again at this point).
Also closes #120 and #119