Rewards refresh enhancements #1780

Closed
opened 2018-07-17 16:25:01 +02:00 by tzarebczan · 2 comments
tzarebczan commented 2018-07-17 16:25:01 +02:00 (Migrated from github.com)

The Issue

  1. To support tiered rewards, we should refresh the reward list after a reward is claimed.
  2. Instead of calling the reward list when going to the Rewards page, call it on the Overview page. This ensures that the user is shown accurately rewards information on Overview.

@tiger5226 can you confirm that the next level in a tiered reward will be available right after the previous level is claimed successfully? I'm assuming they should only see 1 level at a time.

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. - App releases: https://github.com/lbryio/lbry-desktop/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on live chat at https://chat.lbry.io --> ## The Issue 1) To support tiered rewards, we should refresh the reward list after a reward is claimed. 2) Instead of calling the reward list when going to the Rewards page, call it on the Overview page. This ensures that the user is shown accurately rewards information on Overview. @tiger5226 can you confirm that the next level in a tiered reward will be available right after the previous level is claimed successfully? I'm assuming they should only see 1 level at a time. ## System Configuration <!-- For the app, this info is in the About section at the bottom of the Help page. You can include a screenshot instead of typing it out --> <!-- For the daemon, run: curl 'http://localhost:5279' --data '{"method":"version"}' and include the full output --> - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here -->
tiger5226 commented 2018-07-22 18:45:32 +02:00 (Migrated from github.com)

They will see 1 level at a time, and once claimed they will see the next level if the api is called again. This work is ready and should be merged for our internal-apis soon, specifically for many download levels and subscription levels.

They will see 1 level at a time, and once claimed they will see the next level if the api is called again. This work is ready and should be merged for our internal-apis soon, specifically for many download levels and subscription levels.
kauffj commented 2018-07-23 17:46:08 +02:00 (Migrated from github.com)

This shows merged so I'm closing, re-open if I'm missing something.

This shows merged so I'm closing, re-open if I'm missing something.
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/lbry-desktop#1780
No description provided.