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

Update ro.json #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

victorVastory
Copy link

Update ro.json to use count placeholder instead numar

Update ro.json to use count placeholder instead numar
@victorVastory
Copy link
Author

@SathishKumarRajendran @VinothYesuraj @Manikandan-Lakshmanan @Balaji-Elumalai @Mohamed7Imran @Mydeensn Can you review it and merge if it's complaint?
Thank you!

@Mohamed7Imran
Copy link
Contributor

Hi @victorVastory ,

We understand that you are requesting an update to the ro.json file to use "count" as a placeholder instead of "numar" for the Romanian culture. Currently, we have provided "numar" as the placeholder by translating it to Romanian to cater to the localization needs of a diverse user base.

To better understand your specific requirement, could you please provide us with the reason for preferring "count" over "numar"?, However, you can customize the localization to suit your environment by making the necessary changes after cloning it into your project. This allows you to tailor the localization to meet your specific needs.

Regards,
Mohamed

@victorVastory
Copy link
Author

Hi @Mohamed7Imran ,
Thank you for your response.
I've made this merge request due the fact that on romanian language, the placeholder is not recognized for the following version
"@syncfusion/ej2-locale": "20.1.56"
image

@victorVastory
Copy link
Author

Hi @Mohamed7Imran ,
I've made an demo to reproduce the issue on stackblitz, loading the locale with the response coming from syncfusion locale

image

https://stackblitz.com/edit/github-7nqtqa?file=src%2Fapp.component.ts

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