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

Split ArrowheadService class into multiple auto injectable classes #40

Open
BlackRose01 opened this issue Jan 24, 2022 · 1 comment
Open
Labels

Comments

@BlackRose01
Copy link

Hello,

please split the current superclass "ArrowheadService" into different classes relevant to the topic. Currently this class handles all communication and configuration. Methods have long names to distinguish and describe their functionality. At best, each core module should have its own class for Auto Injection. In this way, function names can also be shortened and the readability and understandability of the source code increases. Furthermore, the class does not look so bloated.

KR,
BlackRose01

@borditamas
Copy link
Member

Hello,

before Arrowhead v5.0.0 release we do not plan such a big breaking changes in this lib, but personally I agree with your suggestion.

BR,
Tamás

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants