Skip to content

Apply the polymorpishm for Groups#875

Open
zak39 wants to merge 3 commits intomainfrom
refactor/apply-polymorphism-groups
Open

Apply the polymorpishm for Groups#875
zak39 wants to merge 3 commits intomainfrom
refactor/apply-polymorphism-groups

Conversation

@zak39
Copy link
Collaborator

@zak39 zak39 commented Jul 5, 2023

I applied the polymorphism principle for the main groups for a Workspace (WM- and U-).

I applied the polymorphism principle for the main groups for a Workspace
(WM- and U-).

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>
@zak39 zak39 added backend Modification related to backend refactor labels Jul 5, 2023
@zak39 zak39 requested a review from acdmft July 5, 2023 13:40
@zak39 zak39 self-assigned this Jul 5, 2023
zak39 added 2 commits July 10, 2023 10:51
I moved the group files in the new Groups folder and the namespace that
wear the same name.

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>
I renamed GroupsWorkspaceService to GroupManager and moved in the
lib/Groups folder and its namespace is "OCA\Workspace\Groups" now.

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Modification related to backend refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant