App downloads too many of my contributions, without reason#4916
App downloads too many of my contributions, without reason#4916nicolas-raoul merged 4 commits intocommons-app:masterfrom
Conversation
|
@nicolas-raoul after the change I have made here is the new behaviour. when we open app, Can you please also verify it? Thanks Edit: also when we open app for first time there wil be a 3-4 requests and then it will wait until we scroll Edit: will fix the test. |
nicolas-raoul
left a comment
There was a problem hiding this comment.
This seems to work great! :-)
Codecov Report
@@ Coverage Diff @@
## master #4916 +/- ##
============================================
- Coverage 51.11% 50.99% -0.12%
+ Complexity 2297 2290 -7
============================================
Files 341 341
Lines 15960 15959 -1
Branches 1410 1410
============================================
- Hits 8158 8139 -19
- Misses 7182 7200 +18
Partials 620 620
Continue to review full report at Codecov.
|
Thanks. I have removed unnecessary code from tests as well. can you please check that? |
Description (required)
App downloads all the contributions at once by making lot's of subsequent requests to API. Instead the contributions should be downloaded as users scrolls rather than all at once.
Fixes #4900
What changes did you make and why?
I removed the fetchContributions call inside of onItemAtFrontLoaded . this will prevent function calling when user is not at the end.
Tests performed (required)
prodDebug