Skip to content

Conversation

@Besomhead
Copy link
Contributor

Covers #299

@Besomhead Besomhead changed the title Add List Strings on the Workflow Step method to Workflows API feat: add List Strings on the Workflow Step method to Workflows API Oct 12, 2025
@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.12%. Comparing base (17ec43d) to head (47e037e).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #336      +/-   ##
============================================
+ Coverage     94.10%   94.12%   +0.02%     
- Complexity      883      884       +1     
============================================
  Files           199      199              
  Lines          3082     3090       +8     
  Branches         61       61              
============================================
+ Hits           2900     2908       +8     
  Misses          153      153              
  Partials         29       29              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andrii-bodnar andrii-bodnar linked an issue Oct 13, 2025 that may be closed by this pull request
@andrii-bodnar
Copy link
Member

@Besomhead thank you for the contribution!

I would suggest waiting here until the #335 merge to implement the orderBy according to the new way introduced in that pull request.

@Besomhead
Copy link
Contributor Author

@Besomhead thank you for the contribution!

I would suggest waiting here until the #335 merge to implement the orderBy according to the new way introduced in that pull request.

@andrii-bodnar Sure, let's wait for the merge.

@andrii-bodnar
Copy link
Member

@Besomhead, the #335 PR is merged now.

@Besomhead
Copy link
Contributor Author

@andrii-bodnar nice, updated the solution.

Copy link
Member

@andrii-bodnar andrii-bodnar left a comment

Choose a reason for hiding this comment

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

@Besomhead thank you!

@andrii-bodnar andrii-bodnar merged commit 2c3c706 into crowdin:master Oct 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Workflows API: new List Strings on the Workflow Step method

3 participants