Skip to content

Conversation

@Ravid-A
Copy link
Contributor

@Ravid-A Ravid-A commented Sep 8, 2025

No description provided.

@Ravid-A Ravid-A requested a review from roflmuffin as a code owner September 8, 2025 20:54
roflmuffin
roflmuffin previously approved these changes Oct 12, 2025
@Ravid-A
Copy link
Contributor Author

Ravid-A commented Oct 16, 2025

Will you merge this feature?

@roflmuffin
Copy link
Owner

Will need a quick smoke test once #1012 is in

@Ravid-A
Copy link
Contributor Author

Ravid-A commented Oct 18, 2025

Waiting for the smoke test

@roflmuffin
Copy link
Owner

I meant that you will need to fix the merge conflicts and retest your change.

@Ravid-A
Copy link
Contributor Author

Ravid-A commented Oct 20, 2025

Ohh got you

@Ravid-A
Copy link
Contributor Author

Ravid-A commented Oct 20, 2025

Will need a quick smoke test once #1012 is in

Feature has been tested after conficts have been resolved

@Ravid-A
Copy link
Contributor Author

Ravid-A commented Oct 29, 2025

Will the feature will be considered for future update?
@roflmuffin

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.

4 participants