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

Enhance /api/v1/categories/children Endpoint to Return Parent List #29315

Closed
Tracked by #28493
oidacra opened this issue Jul 19, 2024 · 2 comments · Fixed by #29386
Closed
Tracked by #28493

Enhance /api/v1/categories/children Endpoint to Return Parent List #29315

oidacra opened this issue Jul 19, 2024 · 2 comments · Fixed by #29386

Comments

@oidacra
Copy link
Member

oidacra commented Jul 19, 2024

Parent Issue

#28493

User Story

As a developer, I need the /api/v1/categories/children endpoint to return the parentList of categories when showChildrenCount: true is sent. Alternatively, create a new parameter to handle this. This is necessary to provide the path to the parent category for deeper levels (2-N) to help users navigate back to the selected child category.

Acceptance Criteria

  • Modify the /api/v1/categories/children endpoint to include the parentList when parentList: true is sent.
  • Alternatively, add a new parameter to request the parentList.
  • Ensure the parentList provides the complete path to the parent category for deeper levels (2-N).
  • Update the API documentation to reflect the changes.
  • This is only of 1 Level, only for the children of the inode sent.
  • Write unit tests to cover the new functionality.

Proposed Objective

API Enhancement

Proposed Priority

Priority 2 - Important

dotCMS Version

master

Problem

Screenshot 2024-07-19 at 5 16 05 PM

Request don't have parentList
Screenshot 2024-07-19 at 5 18 26 PM

When we sent filter query param, the endpoint retrieve parentList of the category
Screenshot 2024-07-19 at 5 18 49 PM

Assumptions & Initiation Needs

Assuming the current implementation of the /api/v1/categories/children endpoint can be enhanced to include the parentList. Access to the current codebase and API documentation is required.

Quality Assurance Notes & Workarounds

Quality assurance should verify that the parentList is returned correctly when requested and that the API remains stable. Ensure that edge cases, such as categories with no parents, are handled correctly.

@oidacra oidacra moved this from New to Next 1-3 Sprints in dotCMS - Product Planning Jul 19, 2024
@freddyDOTCMS freddyDOTCMS self-assigned this Jul 19, 2024
@freddyDOTCMS freddyDOTCMS moved this from Next 1-3 Sprints to In Progress in dotCMS - Product Planning Jul 19, 2024
@freddyDOTCMS freddyDOTCMS moved this from In Progress to In Review in dotCMS - Product Planning Jul 30, 2024
@freddyDOTCMS freddyDOTCMS moved this from In Review to Internal QA in dotCMS - Product Planning Aug 2, 2024
@freddyDOTCMS freddyDOTCMS removed their assignment Aug 2, 2024
@dsilvam dsilvam self-assigned this Aug 6, 2024
@dsilvam dsilvam removed their assignment Aug 6, 2024
@dsilvam dsilvam moved this from Internal QA to In Progress in dotCMS - Product Planning Aug 6, 2024
@dsilvam
Copy link
Contributor

dsilvam commented Aug 7, 2024

@dsilvam dsilvam assigned dsilvam and unassigned freddyDOTCMS Aug 7, 2024
@dsilvam dsilvam moved this from In Progress to QA - Backlog in dotCMS - Product Planning Aug 7, 2024
@josemejias11
Copy link
Contributor

Approved: Tested on trunk_a466b5d, Docker, macOS 14.5, FF v126.0.1

@josemejias11 josemejias11 moved this from QA - In Progress to Done in dotCMS - Product Planning Aug 7, 2024
@dsilvam dsilvam closed this as completed Aug 13, 2024
@github-project-automation github-project-automation bot moved this from Done to Internal QA in dotCMS - Product Planning Aug 13, 2024
@dsilvam dsilvam moved this from Internal QA to Done in dotCMS - Product Planning Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants