Skip to content

feat: add Applications API support#267

Merged
andrii-bodnar merged 2 commits into
masterfrom
feature/issue-159_add-applications-api
Jun 8, 2026
Merged

feat: add Applications API support#267
andrii-bodnar merged 2 commits into
masterfrom
feature/issue-159_add-applications-api

Conversation

@innomaxx

Copy link
Copy Markdown
Collaborator

Closes #159

@innomaxx innomaxx requested a review from andrii-bodnar May 31, 2026 13:02
@codecov

codecov Bot commented May 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.33%. Comparing base (f3bca05) to head (cf227af).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #267      +/-   ##
============================================
+ Coverage     93.30%   93.33%   +0.03%     
- Complexity     1840     1846       +6     
============================================
  Files           169      171       +2     
  Lines          4969     4986      +17     
============================================
+ Hits           4636     4653      +17     
  Misses          333      333              

☔ View full report in Codecov by Harness.
📢 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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds support for Crowdin Applications API endpoints for managing custom application data.

Changes:

  • Introduces ApplicationApi with get/add/update/delete/edit methods.
  • Adds ApplicationData model for arbitrary application-defined payloads.
  • Registers the new API service and adds unit tests for API/model behavior.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/CrowdinApiClient/Api/ApplicationApi.php Adds Applications API endpoint wrappers.
src/CrowdinApiClient/Model/ApplicationData.php Adds a model for application data payloads.
src/CrowdinApiClient/Crowdin.php Registers the application service.
tests/CrowdinApiClient/Api/ApplicationApiTest.php Adds tests for Applications API methods.
tests/CrowdinApiClient/Model/ApplicationDataTest.php Adds tests for generic data access on ApplicationData.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/CrowdinApiClient/Api/ApplicationApiTest.php Outdated
Comment thread tests/CrowdinApiClient/Api/ApplicationApiTest.php Outdated
@innomaxx

innomaxx commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator Author

@andrii-bodnar please review and merge

@andrii-bodnar andrii-bodnar merged commit d14cde5 into master Jun 8, 2026
8 checks passed
@andrii-bodnar andrii-bodnar deleted the feature/issue-159_add-applications-api branch June 8, 2026 07:17
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.

Applications API support

3 participants