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
Fix the load-more-button does not display bug #21304
Conversation
this.loadMoreService_.setLoadMoreEnded(); | ||
} | ||
}); | ||
}).then(() => { |
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.
Fix bug.
@@ -79,7 +79,7 @@ | |||
"items": 8, | |||
"left": 5, | |||
"latency": 0, | |||
"url": "/infinite-scroll?items=8&left=2" | |||
"url": "/list/infinite-scroll?items=8&left=2" |
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.
Update routes changes due to #21214.
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.
LGTM thanks : D
* Fix sample * Fix urls after routes refactor * Fix load-more-button doesn't display bug * Fix type check
* Fix sample * Fix urls after routes refactor * Fix load-more-button doesn't display bug * Fix type check
#20993 caused the regression in #21270. This PR reverts the offending change and unskips the relevant broken e2e tests.
Great catch by our e2e tests. xD