Add Rewards Eligibility To Overview Page #530

Closed
MSFTserver wants to merge 1 commit from patch-7 into master
MSFTserver commented 2017-09-04 21:25:00 +02:00 (Migrated from github.com)
Fix For Issue https://github.com/lbryio/lbry-app/issues/529 ![image](https://user-images.githubusercontent.com/18357967/30036838-64629e0a-916b-11e7-8c75-039ec41e165a.png)
kauffj (Migrated from github.com) reviewed 2017-09-04 21:25:00 +02:00
MSFTserver commented 2017-09-05 19:04:43 +02:00 (Migrated from github.com)

not sure of the linking to other pages but heres an example code of adding the link to non eligible part

        {user && user.is_reward_approved
          ? <p>{__("Eligible")}<Icon icon="icon-check" /></p>
          : <a href="#">{__("Eligible")}<Icon icon="icon-ban" /></a>}
not sure of the linking to other pages but heres an example code of adding the link to non eligible part ``` {user && user.is_reward_approved ? <p>{__("Eligible")}<Icon icon="icon-check" /></p> : <a href="#">{__("Eligible")}<Icon icon="icon-ban" /></a>} ```
btzr-io commented 2017-09-05 19:06:05 +02:00 (Migrated from github.com)
I think there is a `link` component, for that... https://github.com/lbryio/lbry-app/blob/master/ui/js/component/link/view.jsx
kauffj commented 2017-09-06 22:13:40 +02:00 (Migrated from github.com)

We are not going to display rewards eligibility on the wallet landing. The rewards area does that, and any improvements to be made here will be done inside that area.

We are not going to display rewards eligibility on the wallet landing. The rewards area does that, and any improvements to be made here will be done inside that area.

Pull request closed

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-desktop#530
No description provided.