Skip to content

Di system services and reminder context#1329

Merged
Futsch1 merged 4 commits intoFutsch1:mainfrom
ThomasKiljanczykDev:di-system-services-and-reminder-context
Mar 27, 2026
Merged

Di system services and reminder context#1329
Futsch1 merged 4 commits intoFutsch1:mainfrom
ThomasKiljanczykDev:di-system-services-and-reminder-context

Conversation

@ThomasKiljanczykDev
Copy link
Copy Markdown
Contributor

@Futsch1 this PR focuses on moving all system services into DI as well as further trimming of the reminder context

@Futsch1
Copy link
Copy Markdown
Owner

Futsch1 commented Mar 27, 2026

@ThomasKiljanczykDev
Thanks for this PR. I think the failed tests are just flaky ones (I have the feeling that the stability decreased slightly since you changed some basic things in the tests, but I might be mistaken...). I will merge as soon as all tests are passed.

If you have the time. please continue to work on DI. Please stop once you have finished the DI refactoring, since I would like to synchronize on how we approach the next steps (let's do this in the discussion thread).

@Futsch1 Futsch1 merged commit fd5ffa9 into Futsch1:main Mar 27, 2026
13 of 18 checks passed
@ThomasKiljanczykDev ThomasKiljanczykDev deleted the di-system-services-and-reminder-context branch March 28, 2026 15:02
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.

2 participants