WAppAI / assistant

A WhatsApp chatbot that leverages Bing AI's and others LLMs conversational capabilities.
MIT License
102 stars 37 forks source link

Sydney should know the name of the users #23

Closed dinushay closed 1 year ago

dinushay commented 1 year ago

On WhatsApp, you can choose a public name. Sydney could take this and address the user by name.

Luisotee commented 1 year ago

This is already planned in #20

dinushay commented 1 year ago

However, it was described as being a feature only for groups

Luisotee commented 1 year ago

Oh I thought you meant for groups. If it's with a private chat you can tell Sydney to remember your name, but she might forget it in the long run. You could enforce her to remember your name by putting it in her context, if I have time I will commit a change to make it work through her context, as using the whatsapp name might be a little more complicated than this.

Luisotee commented 1 year ago

I added it in a new branch called 'beta.' You can set both your name and Sydney's name through the .env file. Please refer to the .env.example for an example. If you come across any bugs, please let us know!

veigamann commented 1 year ago

@Luisotee I think a better approach would be to use WhatsApp's public name as @DinuNuggets mentioned. It could however be optionally overwritten by some !command. That's a bit more user-friendly. Avoids messing with environment variables.

Luisotee commented 1 year ago

This issue is marked as closed since the implementation has been completed by PR #24. However, we are planning to change the implementation in the future. This change will be addressed in a separate issue, as it will not only affect the way Sydney remembers the username but also impact several other aspects.