seratch / ChatGPT-in-Slack

Swift demonstration of how to build a Slack app that enables end-users to interact with a ChatGPT bot
MIT License
441 stars 167 forks source link

Role of bot utterance in thread is always "user" #62

Closed masahiro-hamada closed 1 year ago

masahiro-hamada commented 1 year ago

Description At bolt_listeners.py#L344, the role is always "user".

Issue This static assignment of role potentially limits functionality as it doesn't take into account the actual user roles.

Recommended Fix The role should be assigned using reply["user"] similar to how it's done in bolt_listeners.py#L105-L109. This way, the user's actual role is recognized and utilized.

seratch commented 1 year ago

Thanks for taking the time to report this issue! I'll fix it as soon as I have some free time. If you could send a pull request, that would also be greatly appreciated!

masahiro-hamada commented 1 year ago

Thank you for reviewing. I sent the PR, please check it.

seratch commented 1 year ago

Fixed by #63 ; thanks for your contribution!