Skip to content
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

Mass Action on Retailer should support Select All #36

Open
tuyennn opened this issue Jan 18, 2022 · 0 comments
Open

Mass Action on Retailer should support Select All #36

tuyennn opened this issue Jan 18, 2022 · 0 comments

Comments

@tuyennn
Copy link

tuyennn commented Jan 18, 2022

Preconditions

Magento Version : CE 2.4.x

Module Retailer Version : All version

Environment : Doesn't matter

Third party modules : Doesn't matter

Steps to reproduce

  1. Refer to Invalid argument supplied for foreach() $retailerIds on vendor/smile/module-store-locator/Controller/Adminhtml/Retailer/MassSaveHours.php magento2-module-store-locator#132 with same conditions but should be tested with Retailer Mass Action such as Mass Enable/MassAllowDelivery/MassDelete/MassDisable etc

Expected result

  1. Grid Mass Action should have no problem

Actual result

  1. [Screenshot, logs] Invalid argument supplied for foreach() since retailer_ids parameter doesn't exist with Select All
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant