search

Southbacon / the-crispy

The Crispy, a bot built for the Southbacon Slack community.
MIT License
1 stars 2 forks source link

Giving karma to multiple things shows the same karma for both. #47

Closed eroepken closed 5 years ago

eroepken commented 6 years ago

Describe the bug There’s a small bug in the display logic when you give or take karma from multiple things in a single message. It’s not a bug in the storage of the karma itself, but in the display after the karma is changed.

To Reproduce Steps to reproduce the behavior:

  1. Enter something like @drupal-- @meetings--
  2. Notice that it outputs the same karma for both, when in this case they are vastly different. (@meetings was around -26 and @drupal around -776 as of this post.) Entering the two separately shows the correct karma.

Expected behavior The bot should show the correct karma for each individual thing/user.

Screenshots

screen shot 2018-10-15 at 10 52 26 am
eroepken commented 5 years ago

This was fixed with the code added for #51.