Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Ledger table entries logged based on view and not on time spent #13280

Closed
srirambv opened this issue Feb 23, 2018 · 3 comments
Closed

Ledger table entries logged based on view and not on time spent #13280

srirambv opened this issue Feb 23, 2018 · 3 comments
Assignees
Labels
0.22.x issue first seen in 0.22.x bug feature/rewards

Comments

@srirambv
Copy link
Collaborator

Description

Ledger table entries logged based on view and not on time spent

Steps to Reproduce

  1. Clean install 0.21.12 and enable payments
  2. View few twitch channels ensure they show up in ledger table
  3. Doesn't sort based on time spent but instead lists based on view order

Actual result:
image

Expected result:
Should be sorted based on time spent

Reproduces how often:
100%

Brave Version

about:brave info:

Brave 0.21.12
V8 6.4.388.41
rev 246f900
Muon 5.0.5
OS Release 10.0.16299
Update Channel Beta
OS Architecture x64
OS Platform Microsoft Windows
Node.js 7.9.0
Brave Sync v1.4.2
libchromiumcontent 64.0.3282.140

Reproducible on current live release:
N/A

Additional Information

Reproduced on mac by @LaurenWags

@srirambv srirambv added bug feature/rewards 0.21.x issue first seen in 0.21.x labels Feb 23, 2018
@srirambv srirambv added this to the 0.21.x (Beta Channel) milestone Feb 23, 2018
@NejcZdovc NejcZdovc modified the milestones: 0.21.x (Beta Channel), Triage Backlog Feb 23, 2018
@srirambv
Copy link
Collaborator Author

This is not limited to publisher entry. Happens with other ledger entries as well
image

@bsclifton bsclifton added 0.22.x issue first seen in 0.22.x and removed 0.21.x issue first seen in 0.21.x labels Feb 27, 2018
@arsalankhalid
Copy link
Contributor

arsalankhalid commented Mar 4, 2018

Happy to take a stab :) Already trying to re-create, but I am seeing the ability to sort. However, not in the publisher view, it was just mentioned above that one can re-create this in all types of ledger entries.

mar-04-2018 16-47-41

@NejcZdovc NejcZdovc removed this from the Triage Backlog milestone Apr 6, 2018
@NejcZdovc
Copy link
Contributor

Closing as this will be fixed with #13721 where default sort will be %

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
0.22.x issue first seen in 0.22.x bug feature/rewards
Projects
None yet
Development

No branches or pull requests

4 participants