Skip to content

Misleading log entry when clearing field#2609

Open
Belissimo-T wants to merge 8 commits intoe-valuation:mainfrom
Belissimo-T:feat/2603
Open

Misleading log entry when clearing field#2609
Belissimo-T wants to merge 8 commits intoe-valuation:mainfrom
Belissimo-T:feat/2603

Conversation

@Belissimo-T
Copy link
Collaborator

fixes #2603

Copy link
Member

@richardebeling richardebeling left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

Copy link
Member

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

makes sense, tests so far are good, I have one remark though:

Copy link
Member

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

merge conflict

Copy link
Collaborator

@jooooosef jooooosef left a comment

Choose a reason for hiding this comment

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

lgtm

@niklasmohrin niklasmohrin requested a review from janno42 February 16, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Misleading log entry when clearing field

4 participants