Skip to content

Conversation

@korzol
Copy link
Contributor

@korzol korzol commented Sep 30, 2025

No description provided.

@andrii-bodnar
Copy link
Member

@yevheniyJ could you please also take a look? Thank you!

@andrii-bodnar andrii-bodnar linked an issue Sep 30, 2025 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2025

Codecov Report

❌ Patch coverage is 96.65428% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.10%. Comparing base (faefb3d) to head (c334cd2).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...din/client/core/http/impl/util/RequestEncoder.java 25.00% 3 Missing ⚠️
.../java/com/crowdin/client/core/model/SortOrder.java 40.00% 2 Missing and 1 partial ⚠️
...c/main/java/com/crowdin/client/teams/TeamsApi.java 83.34% 1 Missing and 1 partial ⚠️
...va/com/crowdin/client/core/model/OrderByField.java 88.89% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #335      +/-   ##
============================================
- Coverage     94.39%   94.10%   -0.28%     
- Complexity      846      883      +37     
============================================
  Files           196      199       +3     
  Lines          2847     3082     +235     
  Branches         55       61       +6     
============================================
+ Hits           2687     2900     +213     
- Misses          134      153      +19     
- Partials         26       29       +3     

☔ 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.

@korzol korzol marked this pull request as draft October 2, 2025 15:43
@yevheniyJ yevheniyJ changed the title wip: Initial support for multi-sort feat: Initial support for multi-sort Oct 2, 2025
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.

@korzol thank you for the contribution! 🚀

@andrii-bodnar andrii-bodnar merged commit 17ec43d into crowdin:master Oct 14, 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.

Add the orderBy parameter to list methods

4 participants