Thanks to visit codestin.com
Credit goes to github.com

Skip to content

refactor(api-aco): folder level permissions FLP #4615

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

Open
wants to merge 54 commits into
base: next
Choose a base branch
from

Conversation

leopuleo
Copy link
Contributor

@leopuleo leopuleo commented May 5, 2025

Description

This PR introduces a comprehensive refactoring of the ACO folders, focusing on implementing Folder Level Permissions (FLP). The changes follow clean architecture principles and improve the overall code organisation and maintainability.

Key Changes

Folder Level Permissions (FLP)

  • Implemented new FLP storage operation and CRUD to manage permissions in a separate collection
  • Created new tasks for FLP operations (create, update, delete, sync)
  • Implemented hooks for folder lifecycle events
  • Added use cases for folder access control and permissions management

Folders

  • Handled CRUD operations via separate use cases, together with various decorations (lifecycle events and folder-level permissions)
  • Refactored the admin app by introducing the features: GetFolderHierarchy and ListFoldersByParentIds. These features enable management of the folder tree based on the currently focused folder.

How Has This Been Tested?

Jest + Manually

leopuleo added 30 commits April 16, 2025 16:56
@Pavel910 Pavel910 added this to the 5.43.0 milestone May 5, 2025
@leopuleo leopuleo requested a review from Pavel910 May 8, 2025 07:05
@leopuleo leopuleo self-assigned this May 8, 2025
@leopuleo leopuleo added the aco label May 8, 2025
@leopuleo leopuleo marked this pull request as ready for review May 8, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants