Skip to content

Conversation

@dmtr-p
Copy link
Contributor

@dmtr-p dmtr-p commented Jul 9, 2025

The new endpoint allows users to retrieve pre-translation reports containing language-specific file statistics and pre-translation type information for completed pre-translation jobs.

🤖 Generated with Claude Code

- Add getPreTranslationReport method to Translations class
- Add TypeScript interfaces for PreTranslationReport data models
- Add comprehensive test coverage for the new endpoint
- Resolves crowdin#526

The new endpoint allows users to retrieve pre-translation reports
containing language-specific file statistics and pre-translation type
information for completed pre-translation jobs.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
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.

@dmtr-p thank you for the contribution!

I have just a few suggestions regarding types. Overall, everything looks good to me.

@codecov
Copy link

codecov bot commented Jul 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.00%. Comparing base (9d3440e) to head (978c69a).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #542      +/-   ##
==========================================
+ Coverage   89.99%   90.00%   +0.02%     
==========================================
  Files          39       39              
  Lines        1937     1940       +3     
  Branches      203      203              
==========================================
+ Hits         1743     1746       +3     
  Misses         82       82              
  Partials      112      112              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- Use Method type instead of string for preTranslateType field
- Replace SkippedQaCheckCategories with ProjectsGroupsModel.CheckCategories
- Remove unused SkippedQaCheckCategories interface
- Add ProjectsGroupsModel import for CheckCategories type

Addresses review comments:
- Better type safety using existing Method type
- Use specific CheckCategories interface instead of generic any type

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@andrii-bodnar andrii-bodnar merged commit 08d96f0 into crowdin:master Jul 11, 2025
10 checks passed
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.

Add new Pre-Translation Report endpoint support

2 participants