Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

auto contribute list slow to display in UI after update #2486

Closed
LaurenWags opened this issue Apr 23, 2020 · 5 comments
Closed

auto contribute list slow to display in UI after update #2486

LaurenWags opened this issue Apr 23, 2020 · 5 comments

Comments

@LaurenWags
Copy link
Member

Description:

When testing #2476 and #2477 I noticed that my auto contribute list was slow to display in the UI. Not sure if it is due to the database migration (#2378) or something else.

Steps to Reproduce

  1. Install 1.15
  2. Enable rewards and add sites to ac list so rewards db is populated (I had 5-8 sites each time I tried this, a mix of sites that would and would not become verified in step 6)
  3. Upgrade to 1.15.2
  4. Disconnect network
  5. Launch browser and add custom UA
  6. Relaunch network and open rewards panel and click on refresh status on the existing page
  7. No browser lag or hang while console showed "OnDownload" request
  8. Tap on BAT logo and navigate to Auto Contribute details.
    --> Supported sites says 0. Wait a minute or two (stay on panel), it will update to the expected number
    --> Tap on Supported sites to view the list (to confirm sites on the list have their verified icon as expected per step 2), list is blank. Again, wait a minute or two with panel open, eventually it will update to show your AC list.

Actual result:
AC list is slow to display in the UI

Expected result:
no slowness

Reproduces how often: [Easily reproduced, Intermittent Issue]
easily

Brave Version:
1.15.2(20.04.22.20)

Device details:
iPad 5th Gen w/ iOS12
iPad mini w/ iOS13

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

cc @srirambv as he saw this as well

@kylehickinson
Copy link
Collaborator

@LaurenWags This is because the server publisher list is inserting large batches of publishers into the DB and therefore keeping DB interactions busy. The UI will still be usable, but the data may take a minute or two to show up

@LaurenWags
Copy link
Member Author

gotcha, thanks @kylehickinson - if this is expected I guess we can close this one? wdyt?

@kylehickinson
Copy link
Collaborator

I think possibly the best we could do is maybe add a loader, but given we are looking towards not doing these huge DB inserts in the near future with pub list v4 I think its probably not worth it.

@LaurenWags
Copy link
Member Author

Maybe we leave this open until we get to that point and once v4 is implemented, we can confirm we don't still see this issue?

@kylehickinson
Copy link
Collaborator

Sounds good to me!

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

No branches or pull requests

3 participants