-
Notifications
You must be signed in to change notification settings - Fork 633
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
Progress fetching url limiting #7434
Progress fetching url limiting #7434
Conversation
Batch fetching when more than 50.
I think we've talked informally about this happening, but there's no specific issue about adding these filters. Maybe this issue is relevant? #5041 |
Yeah - that seems related. Not sure it will fix the vue dev tools part or not. |
Also possibly related - #6466 (comment)
|
Codecov Report
|
Hrm - curious, will look into this! |
So, it seems that the state for the "Most popular" subpage is actually this |
Thanks - I'll make that alteration, seems like a decent cleanup. |
Updated! |
Looks good. I think these lines are now dead code if you can clean it up |
Done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Summary
Reviewer guidance
Check that progress loads in a lesson, in a topic, and on the main recommended page and on recommended subpages.
References
I thought there was an issue for this, but I can't find it for the life of me.
Contributor Checklist
PR process:
Testing:
Reviewer Checklist
yarn
andpip
)