Apply Pre-Translation API: add labels support

[andrii-bodnar]

Crowdin has recently added a possibility to Pre-Translate strings by labels. However, the new parameters are not supported in this API client.

The new labelIds and excludeLabelIds parameters should be added to the "Apply Pre-Translation" method in this API client.

API Reference:

• Apply Pre-Translation
• Pre-Translation Status (The attributes field now contains the new fields as well)

[KyleGrande]

Hey! So does this just require an update to the PreTranslateRequest interface or am I misunderstanding?

[andrii-bodnar]

Hi @KyleGrande, yes, it's related to the PreTranslateRequest, PreTranslationStatusAttributes, and probably something else related to the Pre-Translation including Unit tests (needs to be checked in the code).

[KyleGrande]

Thanks @andrii-bodnar for further clarifying. I will look more into it and begin working.

[KyleGrande]

@andrii-bodnar So, I've gone ahead and updated the PreTranslateRequest & PreTranslationStatusAttributes interfaces and now I'm looking to see what related to the unit test should be changed but I am kind of lost. Could you maybe point me in the right direction? I was thinking I needed to add test with labelIds and excludeLabelIds? I am currently looking at the test /crowdin-api-client-js/tests/translations and /crowdin-api-client-js/tests/translationStatus.

[andrii-bodnar]

@KyleGrande I think it would be nice to add a new test case in the tests/translations/api.test.ts

[KyleGrande]

@andrii-bodnar I believe this should be everything #306 , if not please let me know.

Also, thank you very much for guiding me through it.

[andrii-bodnar]

@KyleGrande looks good to me, thanks a lot for the contribution! 🚀