fix: allow claiming rewards with multiple of the same type #22

Merged
neb-b merged 2 commits from rewards-fix into master 2019-01-18 17:26:41 +01:00
neb-b commented 2019-01-17 00:15:58 +01:00 (Migrated from github.com)

This allows multiple rewards of the same type to work. (users with more than one youtube sync channel)

There is still a minor issue of other rewards showing as claiming while you claim a one of multiple rewards with the same type. A larger change will be needed for that to be fixed.

This allows multiple rewards of the same type to work. (users with more than one youtube sync channel) There is still a minor issue of other rewards showing as `claiming` while you claim a one of multiple rewards with the same type. A larger change will be needed for that to be fixed.
skhameneh (Migrated from github.com) reviewed 2019-01-17 00:15:58 +01:00
akinwale (Migrated from github.com) approved these changes 2019-01-18 17:22:31 +01:00
akinwale (Migrated from github.com) left a comment

Looks good.

Looks good.
tiger5226 (Migrated from github.com) reviewed 2019-01-19 05:13:21 +01:00
tiger5226 (Migrated from github.com) left a comment

@seanyesmunt @tzarebczan This should not send it to /reward/new but /reward/claim the /reward/new will be deprecated.

@seanyesmunt @tzarebczan This should not send it to `/reward/new` but `/reward/claim` the `/reward/new` will be deprecated.
neb-b commented 2019-01-19 17:01:32 +01:00 (Migrated from github.com)

Tom said it might be changing. I'll update.

Tom said it might be changing. I'll update.
Sign in to join this conversation.
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/lbryinc#22
No description provided.