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

IBX-8433: Added content tree children filtering #1290

Merged
merged 13 commits into from
Aug 20, 2024

Conversation

ciastektk
Copy link
Contributor

@ciastektk ciastektk commented Jul 16, 2024

🎫 Issue IBX-8433

Related PRs:

Description:

This PR adds ability to filter Content Tree children (subitems) by Criteria passed by filter query param.

For QA:

Request examples:

  • No filter: GET http://127.0.0.1:8000/api/ibexa/v2/location/tree/load-subitems/1/30/0

  • Filter by content type identifier folder: GET http://127.0.0.1:8000/api/ibexa/v2/location/tree/load-subitems/69/30/0?filter[ContentTypeIdentifierCriterion]=folder

  • Filter by content type identifier folder AND image: GET http://127.0.0.1:8000/api/ibexa/v2/location/tree/load-subitems/69/30/0?filter[ContentTypeIdentifierCriterion][0]=folder&filter[ContentTypeIdentifierCriterion][1]=image

  • Filter by content type identifier image OR subtree user group: GET http://127.0.0.1:8000/api/ibexa/v2/location/tree/load-subitems/1/10/0?filter[OR][ContentTypeIdentifierCriterion]=image&filter[OR][SubtreeCriterion]=/1/5/

Documentation:

@ciastektk ciastektk changed the base branch from main to 4.6 July 16, 2024 10:26
@ciastektk ciastektk changed the title Ibx 8433 added content tree children filtering IBX-8433: Added content tree children filtering Jul 16, 2024
@ciastektk ciastektk force-pushed the ibx-8433-added-content-tree-children-filtering branch 7 times, most recently from 79b1f47 to 606d9e8 Compare July 18, 2024 09:11
@ciastektk ciastektk marked this pull request as ready for review July 18, 2024 11:16
@ciastektk ciastektk requested a review from a team July 18, 2024 11:45
@konradoboza konradoboza requested a review from a team July 19, 2024 09:01
@ciastektk ciastektk requested a review from konradoboza July 19, 2024 10:37
@konradoboza konradoboza requested a review from a team July 19, 2024 10:52
@ciastektk ciastektk force-pushed the ibx-8433-added-content-tree-children-filtering branch from 56166ad to ed285d2 Compare August 9, 2024 05:45
@ciastektk ciastektk force-pushed the ibx-8433-added-content-tree-children-filtering branch from ed285d2 to 560a050 Compare August 9, 2024 08:41
@ciastektk ciastektk requested review from adamwojs and a team August 12, 2024 10:55
Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on IbexaDXP 4.6 commerce.

@ciastektk ciastektk force-pushed the ibx-8433-added-content-tree-children-filtering branch from 1bdbdcf to b19d7a1 Compare August 20, 2024 12:12
Copy link

@ciastektk ciastektk merged commit 471396a into 4.6 Aug 20, 2024
29 checks passed
@ciastektk ciastektk deleted the ibx-8433-added-content-tree-children-filtering branch August 20, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants