Updated doHelp() & Added Self Tip Detection #8

Merged

ProfessorDey merged 2 commits from patch-2 into master

2018-01-04 01:28:56 +01:00

ProfessorDey commented

2017-12-25 20:24:08 +01:00

(Migrated from github.com)

Updated doHelp() to match updated module description in Commit ec48f11 (ec48f114d8 (diff-baad512c11a603dedf1cf7f391e341bd))
Also altered help logic so the help message is always DM'd to user to assist new users who struggle to find how to begin a dm session and to improve general usability.

Further added a self tip check to sendLbc() to prevent duplicate messages being received by the user, so as to make clear that the tip is only being sent once.

Updated doHelp() to match updated module description in Commit ec48f11 (https://github.com/lbryio/lbry-tipbot/commit/ec48f114d87feb77e4b6856ca6599659b51f201d#diff-baad512c11a603dedf1cf7f391e341bd) Also altered help logic so the help message is always DM'd to user to assist new users who struggle to find how to begin a dm session and to improve general usability. Further added a self tip check to sendLbc() to prevent duplicate messages being received by the user, so as to make clear that the tip is only being sent once.

nikooo777 commented

2018-01-03 01:24:51 +01:00

(Migrated from github.com)

by merging the other PR this is now conflicting, would you mind adjusting that real quick? i'll merge that afterwards

ProfessorDey commented

2018-01-03 23:02:43 +01:00

(Migrated from github.com)

There you go @nikooo777, that should now match up properly, the fact github doesn't offer a clean way to update PRs to match master is a bit of a pain, but yay for multiple tabs XP

Anyway, this should be just a quick and easy merge now.

There you go @nikooo777, that should now match up properly, the fact github doesn't offer a clean way to update PRs to match master is a bit of a pain, but yay for multiple tabs XP Anyway, this should be just a quick and easy merge now.

😆 1

nikooo777 commented

2018-01-04 01:29:08 +01:00

(Migrated from github.com)

thanks a lot :)

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#8

No description provided.