Skip to content

Conversation

@bashofmann
Copy link
Contributor

@bashofmann bashofmann commented Sep 8, 2025

No description provided.

@bashofmann bashofmann requested a review from a team as a code owner September 8, 2025 10:58
@bashofmann bashofmann marked this pull request as draft September 8, 2025 11:00
@bashofmann bashofmann force-pushed the feat/bashofmann/additional-status-fields branch 2 times, most recently from aaac391 to bb9a088 Compare September 18, 2025 13:19
@bashofmann bashofmann marked this pull request as ready for review September 18, 2025 13:20
Copy link
Contributor

@Robert-Stam Robert-Stam left a comment

Choose a reason for hiding this comment

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

IMO it needs documentation (comments)

@bashofmann
Copy link
Contributor Author

Docs: yes, I wanted to add the fields first and have a discussion about this, and if everything is ok add the docs afterwards. Before I waste time with doing docs for things we are going to remove again :-)

Will work on this next.

@bashofmann bashofmann force-pushed the feat/bashofmann/additional-status-fields branch from 5689b27 to c0a6e4f Compare September 25, 2025 13:50
@bashofmann bashofmann force-pushed the feat/bashofmann/additional-status-fields branch from c0a6e4f to 83e8f38 Compare November 24, 2025 17:38
@bashofmann bashofmann force-pushed the feat/bashofmann/additional-status-fields branch from 83e8f38 to 627f87d Compare November 25, 2025 14:29
Copy link
Contributor

@Robert-Stam Robert-Stam left a comment

Choose a reason for hiding this comment

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

LGTM, Let's see if @hossainemruz has remarks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants