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

Added 'zh' into 'framework/messages/config.php' #19995

Merged
merged 2 commits into from
Oct 18, 2023

Conversation

NabiKAZ
Copy link
Contributor

@NabiKAZ NabiKAZ commented Oct 7, 2023

The zh directory there is in the framework/messages path, but it was not added in the config.php file. That I did. (maybe the zh folder needed to be deleted, which needs to be checked)

@what-the-diff
Copy link

what-the-diff bot commented Oct 7, 2023

PR Summary

  • Additional Language Support in the Framework
    The framework's messaging system has now added support for a new language, zh (Chinese). This means the app can now communicate more effectively with users who prefer Chinese.

@codecov
Copy link

codecov bot commented Oct 7, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (53f1619) 48.94% compared to head (643a311) 48.96%.
Report is 16 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #19995      +/-   ##
==========================================
+ Coverage   48.94%   48.96%   +0.02%     
==========================================
  Files         445      445              
  Lines       42817    42839      +22     
==========================================
+ Hits        20956    20978      +22     
  Misses      21861    21861              
Files Coverage Δ
framework/messages/config.php 0.00% <0.00%> (ø)

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@samdark
Copy link
Member

samdark commented Oct 9, 2023

Fallback should've covered it already. Did you have any problems with the current master config?

@NabiKAZ
Copy link
Contributor Author

NabiKAZ commented Oct 10, 2023

As you can see, I have changed only one file. I didn't test the overall configuration of the framework and I don't know why it gave missing coverage!

@samdark
Copy link
Member

samdark commented Oct 12, 2023

I mean functionality, not coverage. When zh-CN isn't found, it falls back to zh automatically.

@rob006
Copy link
Contributor

rob006 commented Oct 12, 2023

@samdark This is config adjustment for #19832, right now command for extracting messages in framework will ignore zh directory.

@NabiKAZ I that we should only rename zh-CN to zh to match current directory structure introduced by #19832

@NabiKAZ
Copy link
Contributor Author

NabiKAZ commented Oct 17, 2023

I did that, now we only have zh and zh-TW in config and directories.

@samdark samdark merged commit 5d8a96e into yiisoft:master Oct 18, 2023
48 of 49 checks passed
@samdark
Copy link
Member

samdark commented Oct 18, 2023

Thanks.

@samdark samdark added this to the 2.0.50 milestone Oct 18, 2023
samdark added a commit that referenced this pull request Oct 19, 2023
* added pcntl to requirements check

* Fix #13920: Fixed erroneous validation for specific cases

* Fix #13920: Added my name to CHANGELOG.md

* trim(): Passing null to parameter #1 ($string) of type string is deprecated

* Added section about Unsafe Reflection in Security best practices doc (#19948)

Co-authored-by: Bizley <[email protected]>

* Update CHANGELOG.md

* Fixed tests.

* Fix #13920: Add unit test

* Update `bower-asset/inputmask`, `bower-asset/punycode`.

* added CHANGELOG line

* Fix order.

* fix: #19978 - Mistake in Korean translation

* fix: keep doublequote

* Do not duplicate log messages in memory

* Update framework/log/FileTarget.php

Co-authored-by: Bizley <[email protected]>

* Update concept-di-container.md

Сслка "Конфигурация приложения" была не верной, она вела на страницу "Service-locator'a"

* Update CHANGELOG.md

* release version 2.0.49.1

* prepare for next release

* Revert changes in `mimeTypes.php` from 4a1f2c6

restores #19936

* update actions/checkout to v4

* Fix mime type generator

* Added note

* Update structure-controllers.md (#20003)

добавил пропущенное слово "как"

* Update CHANGELOG.md

2.0.49.2 changelog

* release version 2.0.49.2

* prepare for next release

* New methods: BaseActiveRecord::loadRelations() and BaseActiveRecord::loadRelationsFor().

* Fixed a bug where the yii serve command would break if a custom router was supplied and it had a space in the path

* Fix `MaskedInputAsset::class`.

* Fix #19927: Fixed `console\controllers\MessageController` when saving translations to database: fixed FK error when adding new string and language at the same time, checking/regenerating all missing messages and dropping messages for unused languages

* Added 'zh' into 'framework/messages/config.php' (#19995)

---------

Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Alexander Makarov <[email protected]>
Co-authored-by: Oleg Poludnenko <[email protected]>
Co-authored-by: Bizley <[email protected]>
Co-authored-by: Sonia Zorba <[email protected]>
Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Akbar Herlambang <[email protected]>
Co-authored-by: lubosdz <[email protected]>
Co-authored-by: Yuriy Bachevskiy <[email protected]>
Co-authored-by: Robert Korulczyk <[email protected]>
Co-authored-by: salehhashemi1992 <[email protected]>
Co-authored-by: PowerGamer1 <[email protected]>
Co-authored-by: Brad Bell <[email protected]>
Co-authored-by: Alexandru Trandafir Catalin <[email protected]>
Co-authored-by: Nabi KaramAliZadeh <[email protected]>
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.

3 participants