Tipbot Repairs #36

Merged
QuirkyRobots merged 1 commit from patch-5 into master 2018-10-10 17:22:11 +02:00
QuirkyRobots commented 2018-10-09 14:14:12 +02:00 (Migrated from github.com)
  • Formatting corrections: URL contained markdown which is only supported in embed: {}
  • Duplicate command removed: !tip and !tips now display the same message so only one required.
  • Duplicate tipper name removed: The bot mentioned the tippers name twice concurrently.
  • Removed unnecessary code: Unnecessary code removed ${message.content.split(' ', 1)[0]}\
  • Removed author: Author was set as !tip and not required.
- Formatting corrections: URL contained markdown which is only supported in `embed: {}` - Duplicate command removed: `!tip` and `!tips` now display the same message so only one required. - Duplicate tipper name removed: The bot mentioned the tippers name twice concurrently. - Removed unnecessary code: Unnecessary code removed `${message.content.split(' ', 1)[0]}\` - Removed author: Author was set as `!tip` and not required.
ykris45 (Migrated from github.com) approved these changes 2018-10-09 14:23:14 +02:00
ykris45 (Migrated from github.com) left a comment

thanks electron

thanks electron
filipnyquist commented 2018-10-10 17:22:06 +02:00 (Migrated from github.com)

LGTM!

LGTM!
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-tipbot#36
No description provided.