Skip to content

Conversation

@mjamilasfihani
Copy link
Contributor

I just realized that the EN locale is using "The" *notice the "T" is in uppercase.
So, if we use ID locale it will use the field name instead.

i.e :

from The phone number is not valid. become Nomor telepon tidak valid.


hi! @milwad-dev I got the cs-fixer from my GH Actions, because of that I will drafted this PR.

@milwad-dev milwad-dev changed the title Update validate.php | ID Locale [1.x] Update validate.php | ID Locale Feb 3, 2025
@milwad-dev
Copy link
Owner

I can't understand why you changed:attribute to :Attribute, and what is the result?

@mjamilasfihani
Copy link
Contributor Author

hi! uh yeah sorry, based on the Laravel documentation https://laravel.com/docs/11.x/localization#replacing-parameters-in-translation-strings when we define :Attribute the first letter will be capitalized, so if we use ID as the locale, the name attribute when it fails will be transformed automatically.. since in ID localization, word "The" is actually "as is", so we remove "The" in "The :attribute ..." to be ":Attribute ..." instead. if my statement isn't clear yet, please tell me. thank you.

Copy link
Owner

@milwad-dev milwad-dev left a comment

Choose a reason for hiding this comment

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

Thanks!

@milwad-dev milwad-dev marked this pull request as ready for review February 4, 2025 05:36
@milwad-dev milwad-dev merged commit 33fd723 into milwad-dev:1.x Feb 4, 2025
19 checks passed
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.

2 participants