MachUpskillingFY17 / JabbR-Core

Modern edition of JabbR chat client using .NET Core
MIT License
7 stars 8 forks source link

Adina/gravatar command #325

Closed ashanhol closed 7 years ago

ashanhol commented 7 years ago

Addresses #228. Test changing your gravatar with /gravatar [emailAddress]. There should be a grey text pop-up notifying you you've changed your gravatar, and the change should be visible in your Account settings and in the side bar displaying users in room. There is an issue with it displaying your gravatar on recent messages sent in the chatroom, but if you leave and come back it should display your updated gravatar on previously sent messages. It seems like this is a context issue because the client.hash is getting updated in the database for the user. Will submit bug on backlog.

kevinleung23 commented 7 years ago

@ashanhol GravatarCommand works correctly with the email parameter. Account page shows gravatar and works as described in PR description. :shipit: