Skip to content

Conversation

@andreichirila0712
Copy link
Contributor

@andreichirila0712 andreichirila0712 commented Dec 9, 2025

This adds support for String/Asset Comments API. (closes: #347)

Changes:

  • Created an additional class: Attachment.class to model the attachment;
  • StringComment.class and AddStringCommentRequest.class classes have been modified to contain a list of attachments;
  • Implemented the method to delete an attachment from a string comment;
  • Fixed a typo in the JavaDoc comment of the batch operations method (PATCH) to correctly redirect the user to the documentation;
  • Response files (JSON) have been modified to take into account the new attachments field
  • Added unit-testing coverage for added method, and for existing ones additional assertions to ensure correct functionality.

Notes:

  • Any feedback regarding the implementation or if there is something that is not up to the requirements would be appreciated!

* Added an additional class to model the attachment;
* Modified existing classes: StringComment and AddStringCommentRequest to contain a list of attachments;
* Implemented the method to delete an attachment from a string comment;
* Fixed a typo in the batch operations method (PATCH) to correctly redirect the user to documentation;
* Modified JSON response files to mirror the attachment addition;
* Tested the newly added method and added additional asserts in existing tests to ensure correct functionality after adding attachments;
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.29%. Comparing base (4b25348) to head (1981038).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #352      +/-   ##
============================================
+ Coverage     93.28%   93.29%   +0.01%     
- Complexity      903      904       +1     
============================================
  Files           202      202              
  Lines          3258     3261       +3     
  Branches         70       70              
============================================
+ Hits           3039     3042       +3     
  Misses          183      183              
  Partials         36       36              

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

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.

@andreichirila0712 thank you!

@andrii-bodnar andrii-bodnar merged commit 550e91f into crowdin:master Dec 9, 2025
3 checks passed
@andreichirila0712 andreichirila0712 deleted the feature/347-string-comment-attachments branch December 9, 2025 10:38
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 support for String Comment Attachments

3 participants