Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add Strings Exporter Settings API #147 #153

Merged

Conversation

faizan1191
Copy link
Contributor

This pull request will add Crowdin Strings Exporter settings API endpoints in the PHP API client.

Fixes: #147

@faizan1191 faizan1191 changed the title Feat: Add Strings Exporter Settings API #147 feat: Add Strings Exporter Settings API #147 Oct 1, 2023
@andrii-bodnar
Copy link
Member

@faizan1191 could you please check the failed pipeline? Thank you!

@andrii-bodnar andrii-bodnar added the hacktoberfest This issue welcomes contributions for Hacktoberfest label Oct 3, 2023
@faizan1191
Copy link
Contributor Author

@andrii-bodnar sure i am checking it.

@faizan1191
Copy link
Contributor Author

@andrii-bodnar just to acknowledge you I am on it and will fix this by today.

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.

@faizan1191 thank you!

@andrii-bodnar
Copy link
Member

@faizan1191 some Unit tests have failed, could you please take a look?

@faizan1191
Copy link
Contributor Author

@andrii-bodnar allow me sometime i will fix this by today

@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #153 (3318161) into master (a53a35f) will decrease coverage by 0.02%.
Report is 7 commits behind head on master.
The diff coverage is 78.95%.

@@             Coverage Diff              @@
##             master     #153      +/-   ##
============================================
- Coverage     90.41%   90.39%   -0.02%     
- Complexity     1181     1201      +20     
============================================
  Files           107      111       +4     
  Lines          3076     3152      +76     
============================================
+ Hits           2781     2849      +68     
- Misses          295      303       +8     
Files Coverage Δ
...CrowdinApiClient/Api/StringsExporterSettingApi.php 100.00% <100.00%> (ø)
src/CrowdinApiClient/Crowdin.php 100.00% <ø> (ø)
.../CrowdinApiClient/Model/StringsExporterSetting.php 65.22% <65.22%> (ø)

... and 4 files with indirect coverage changes

@andrii-bodnar andrii-bodnar merged commit 0808bfa into crowdin:master Oct 5, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest This issue welcomes contributions for Hacktoberfest hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strings Exporter Settings API
2 participants