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

feat: Function implementation: manage and update pump state (XD-65) #198

Merged
merged 14 commits into from
Jun 26, 2024

Conversation

KonsumGandalf
Copy link
Collaborator

Closes: #65

apps/backend/src/app/app.module.ts Outdated Show resolved Hide resolved
apps/backend/src/app/config/validation.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@IngoSternberg IngoSternberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have to say I couldn't really review the changes properly because there are so many files changed (because of the rebasing) and multiple commits. But from what I saw you added the 3 states which is nice and your pump-analyse.utils.ts and the test for it look really fancy and good.

I am sorry about the failing tests on unchanged files. I did not know anything about that. I think since it is not just that one test we might need an issue for that to fix all failing tests on develop.

On the files changed list there are a lot of things different to the current state of develop (f.e. my #191 is not there) so that still needs to be brought back. Maybe consider creating a new branch and putting your changes there (I did the same for #82 and with intellij it was actually not a lot of effort. I just clicked "show diff with working tree" on the old branch and put all files I wanted from there over)

@IngoSternberg
Copy link
Contributor

I created an issue for the failing tests #200 and already started working on it

HaruspexSan and others added 11 commits June 26, 2024 01:00
Co-authored-by: Patrick Schmidt <[email protected]>
Co-authored-by: Ingo Sternberg <[email protected]>
Signed-off-by: Maximilian Krug <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
Signed-off-by: David Schmidt <[email protected]>
@KonsumGandalf KonsumGandalf merged commit 0905eeb into develop Jun 26, 2024
4 checks passed
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.

Function implementation: manage and update pump state
5 participants