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

Ledger table doesnt auto populate the publishers #11273

Closed
srirambv opened this issue Oct 4, 2017 · 5 comments
Closed

Ledger table doesnt auto populate the publishers #11273

srirambv opened this issue Oct 4, 2017 · 5 comments
Assignees
Labels

Comments

@srirambv
Copy link
Collaborator

srirambv commented Oct 4, 2017

Test plan

#11592 (comment)


Description

Ledger table doesnt auto populate the publishers

Steps to Reproduce

  1. Clean install 0.19.28
  2. enable payments
  3. keep the payments tab open
  4. Visit brianbondy.com in a new tab
  5. Wait for a minute,
  6. switch to payments tab,
  7. publisher is not shown in ledger table
  8. Reload the preferences page
  9. publisher is shown in the ledger table
  10. Visit clifton.io in a new tab,
  11. stay on the tab for a minute,
  12. switch to preferences tab,
  13. publisher not shown,
  14. reload the preferences tab,
  15. publisher shows up

Actual result:

autoincludepuubs

Expected result:
Should auto-refresh ledger table when criteria is met

Reproduces how often: [What percentage of the time does it reproduce?]
100%

Brave Version

about:brave info:

Brave 0.19.28
rev 08781a5
Muon 4.4.25
libchromiumcontent 61.0.3163.100
V8 6.1.534.41
Node.js 7.9.0
Update Channel Beta
OS Platform Microsoft Windows
OS Release 10.0.15063
OS Architecture x64

Reproducible on current live release:

No

Additional Information

cc: @LaurenWags @kjozwiak

@NejcZdovc
Copy link
Contributor

@srirambv can you please open new tab and then switch to preference tab?

@NejcZdovc NejcZdovc added this to the 0.19.x (Beta Channel) milestone Oct 4, 2017
@srirambv
Copy link
Collaborator Author

srirambv commented Oct 4, 2017

No change in opening a new tab and switching to preferences tab. Either the tab or preferences tab need to be refreshed in order for the ledger table to show the publisher

@LaurenWags
Copy link
Member

I have run into this as well, but not consistently. I've found that my table is more likely to be updated when opening a new tab as you mention here @NejcZdovc #11273 (comment) but not always.

@bsclifton bsclifton added the priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). label Oct 5, 2017
@bsclifton
Copy link
Member

Moving to 0.19.x release 2

@kjozwiak
Copy link
Member

@NejcZdovc, can we remove the PR/pending-review label as it looks like the #11592 PR has been reviewed and merged?

QA: Test cases can be found under #11592 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

7 participants