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

464 fix filtering objects or variant in grid tab #465

Conversation

lucfumagalli
Copy link
Contributor

ISSUE 464: There is no way to filter objects in grid views based on whether they are of variant type or not

This PR is related to issue #464.

I resolved the issue inserting a select ComboBox in the toolbar of objects grid view. Now you can select what type of objects you can show.

Copy link

github-actions bot commented Mar 15, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@wisconaut wisconaut added this to the 1.4.1 milestone Mar 15, 2024
@lucfumagalli
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@lucfumagalli
Copy link
Contributor Author

I just notice that when you try to filtering object via grid column the list doesn't update. Perhaps you have some suggestion or you can check better than me.

@lucfumagalli
Copy link
Contributor Author

Now it works great!

src/Helper/GridHelperService.php Outdated Show resolved Hide resolved
translations/admin.en.yaml Outdated Show resolved Hide resolved
src/Helper/GridHelperService.php Outdated Show resolved Hide resolved
@lucfumagalli lucfumagalli requested a review from kingjia90 March 21, 2024 08:04
Copy link
Contributor

@kingjia90 kingjia90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for applying the changes, i have some further inputs.
Other than that, the rest looks good to me and it would be ready to be merged.

public/js/pimcore/object/helpers/gridTabAbstract.js Outdated Show resolved Hide resolved
src/Helper/GridHelperService.php Outdated Show resolved Hide resolved
src/Helper/GridHelperService.php Outdated Show resolved Hide resolved
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@lucfumagalli
Copy link
Contributor Author

All requests have been pushed. Are we now ready to merge?
Thank you for your support.

@kingjia90
Copy link
Contributor

Yes, it's merged now

Grazie ancora per il tuo contributo e ti auguro un buon finesettimana 👍🏻

@kingjia90 kingjia90 merged commit d4fc52d into pimcore:1.x Mar 22, 2024
8 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 22, 2024
@lucfumagalli lucfumagalli deleted the 464-fix-filtering-objects-or-variant-in-grid-tab branch March 22, 2024 09:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.