Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Voter Info doesn't sort after first use #439

Closed
tobias-g1 opened this issue Jan 16, 2019 · 1 comment
Closed

[BUG] Voter Info doesn't sort after first use #439

tobias-g1 opened this issue Jan 16, 2019 · 1 comment
Assignees
Labels
bug Something isn't working
Milestone

Comments

@tobias-g1
Copy link

Describe the bug
The voter info table does not sort after the first use, you have the ability to sort the table once, however, when a user attempts to sort again, this does not work.

To Reproduce
Steps to reproduce the behaviour:

  1. Go to your feed and find any post that has had 5 or more votes.
  2. Click on voter icon shown on the bottom of the postcard
  3. Note that when you enter the page that the table is sorted by rewards, the highest STU contributions are shown at the top.
  4. Click on the sort option shown at the top left of the page, you will be presented with three options (rewards, per cent, time).
  5. Click on per cent.
  6. Note that the table now resorts to show the voters with the highest vote percentage.
  7. Repeat step 4 and select rewards.
  8. Note that the data in the table does not resort.

Expected behavior
The table data shown in the voter info pages should reflect the sort method chosen by a user.

Screenshots or Video
sort_order

Smartphone (please complete the following information):
OS: Android 8.0.1
Version 2.0.6

@tobias-g1 tobias-g1 added the bug Something isn't working label Jan 16, 2019
@feruzm feruzm added this to the week6 milestone Jan 16, 2019
@feruzm
Copy link
Member

feruzm commented Jan 16, 2019

good catch, thanks @tobias-g1 for reporting! we will work on this in future updates 👍

@ue ue self-assigned this Jan 23, 2019
@ue ue closed this as completed Jan 23, 2019
ue added a commit that referenced this issue Jan 25, 2019
added render require for force render when data changed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants