ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 60 forks source link

Unset OpenAI Key Error #770

Open 0x4007 opened 12 months ago

0x4007 commented 12 months ago
          Can you change the error to be like? 
! start with a bang

If this is not set or is an empty string then the OpenAI API will throw an error by default anyway, should I still implement null and throw our own error?

It would provide clarity to our early users if we explain how to enable this feature. Basically it should say something like

"You must configure the openai-api-key property in the bot configuration in order to use AI powered features."

_Originally posted by @pavlovcik in https://github.com/ubiquity/ubiquibot/pull/748#discussion_r1326295811_

0x4007 commented 12 months ago

@keyrxng this is for you

Sadaf-A commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago

Too many assigned issues, you have reached your max of 2

Sadaf-A commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago

Too many assigned issues, you have reached your max of 2

Keyrxng commented 11 months ago

already done just as part of /ask, will cherry pick now that /ask has been merged

(idk why I tagged already lmao)

Sadaf-A commented 11 months ago

@already done just as part of /ask, will cherry pick now that /ask has been merged

My bad, I didn't see any PR referenced by you so thought it was up for grabs. thanks for telling I'll pick something else!

Keyrxng commented 11 months ago

already done just as part of /ask, will cherry pick now that /ask has been merged

My bad, I didn't see any PR referenced by you so thought it was up for grabs. thanks for telling I'll pick something else!

that's my mistake should have noted this my bad dude