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

Documents Bulk actions for Grid component #244

Merged
merged 5 commits into from
Apr 15, 2019

Conversation

sarjon
Copy link
Contributor

@sarjon sarjon commented Apr 10, 2019

No description provided.

@matks
Copy link
Contributor

matks commented Apr 12, 2019

👏

Do you think it would be relevant here to quickly explain how BulkActions require a dedicated Controller to handle the route they link to (for example SubmitBulkAction has a 'submit_route' option that needs to be given a valid SF route name to handle the submit process) ?

Or maybe that's for another PR

@sarjon
Copy link
Contributor Author

sarjon commented Apr 15, 2019

I think that should be part of another section. As this is required for other actions as well.

matks and others added 3 commits April 15, 2019 10:32
Co-Authored-By: sarjon <jonusas.sarunas@gmail.com>
Co-Authored-By: sarjon <jonusas.sarunas@gmail.com>
@sarjon
Copy link
Contributor Author

sarjon commented Apr 15, 2019

@matks addressed all your suggestions.

@matks
Copy link
Contributor

matks commented Apr 15, 2019

Thank you @sarjon

@matks matks merged commit 62d1e99 into PrestaShop:master Apr 15, 2019
@sarjon sarjon deleted the grid/bulk-actions branch April 15, 2019 09:12
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

Successfully merging this pull request may close these issues.

3 participants