Skip to content

Refactor/oauth2 api controller#60487

Open
icarta-l wants to merge 1 commit into
nextcloud:masterfrom
icarta-l:refactor/oauth2-api-controller
Open

Refactor/oauth2 api controller#60487
icarta-l wants to merge 1 commit into
nextcloud:masterfrom
icarta-l:refactor/oauth2-api-controller

Conversation

@icarta-l
Copy link
Copy Markdown

Summary

This pull request aims at discussing the redesign of the architecture of the nextcloud/server repository. It follows the Clean Architecture principles by Robert C. Martin and the hybrid packaged by components and features approach.

TODO

  • Have the class diagrams reviewed and discussed by nextcloud/server repository maintainers
  • Write the code accordingly to the class diagrams

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@icarta-l icarta-l requested a review from a team as a code owner May 17, 2026 04:45
@icarta-l icarta-l requested review from Altahrim, artonge, come-nc and icewind1991 and removed request for a team May 17, 2026 04:45
Write GetOauth2TokenController.md, GetOauth2TokerFeature.md,
Oauth2Domain and readme.md files to present a new architecture of
the token retrieval feature. Files can be found under a new
"diagrams" folder inside the oauth2/lib repository.

Signed-off-by: Idan <cartaidan@gmail.com>
@icarta-l icarta-l force-pushed the refactor/oauth2-api-controller branch from 290f0d6 to fbcaa3b Compare May 17, 2026 07:57
@icarta-l
Copy link
Copy Markdown
Author

I have moved the interface for the repositories and other dependencies passed to the Use Case in the GetOauth2TokenInternals/GetOauth2TokenUseCase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The software architecture miss coherence and structure

1 participant