Skip to content

Streamline component management - unified access via service_container #699

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

Closed
vdusek opened this issue Nov 15, 2024 · 0 comments · Fixed by #691
Closed

Streamline component management - unified access via service_container #699

vdusek opened this issue Nov 15, 2024 · 0 comments · Fixed by #691
Assignees
Labels
debt Code quality improvement or decrease of technical debt. t-tooling Issues with this label are in the ownership of the tooling team. v0.5

Comments

@vdusek
Copy link
Collaborator

vdusek commented Nov 15, 2024

Description

The current setup has significant complexity due to multiple ways of passing components like Configuration, EventManager, and StorageClient. This leads to numerous edge cases and unexpected behavior, making the system harder to maintain and use consistently.

Related issues

Solution

  • Standardize access to Configuration, EventManager, and StorageClient exclusively through the service_container module.
  • This will introduce a breaking change.
@vdusek vdusek added the t-tooling Issues with this label are in the ownership of the tooling team. label Nov 15, 2024
@vdusek vdusek self-assigned this Nov 18, 2024
@vdusek vdusek added this to the 103rd sprint - Tooling team milestone Nov 18, 2024
@vdusek vdusek added debt Code quality improvement or decrease of technical debt. v0.5 labels Nov 26, 2024
@vdusek vdusek closed this as completed in 1d31c6c Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debt Code quality improvement or decrease of technical debt. t-tooling Issues with this label are in the ownership of the tooling team. v0.5
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant