-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Issue 24400 #25365
Issue 24400 #25365
Conversation
Hi @prabhatrawat-webkul. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
@DrewML Can you please let me know why the static test failed. |
@magento run all tests |
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.
Hi @prabhatrawat-webkul! Thank you for your contribution. Changes requested.
|
||
if (this.isPageSelected() && selected !== total) { | ||
return this.selectPage(); | ||
} |
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.
Keyword return
at line 244 should have an empty line above it.
|
||
if (total < selected && total > -1) { | ||
total = selected; | ||
} | ||
if (this.excludeMode()) { |
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.
Keyword if
should have an empty line above it at line 325.
@prabhatrawat-webkul, I am closing this PR now due to inactivity. |
Hi @prabhatrawat-webkul, thank you for your contribution! |
Description (*)
Fixed Issues (if relevant)
Manual testing scenarios (*)
Expected Result is :
https://prnt.sc/p09qof
Questions or comments
Contribution checklist (*)