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

Improve Arabic & convert to arabic numbers #198

Merged
merged 2 commits into from
Jun 6, 2021

Conversation

Misioka
Copy link

@Misioka Misioka commented May 26, 2021

In some cases like e.g. react-native Number.toLocaleString is not working. That's why I have added the convert function to the dictionary.

@EvanHahn
Copy link
Owner

Does npm test pass after this change? Should you update the Arabic test CSV file?

@Misioka
Copy link
Author

Misioka commented May 29, 2021

@EvanHahnI have updated CSV and now all tests passed.

@EvanHahn
Copy link
Owner

Thanks! I'll take a look soon.

@EvanHahn EvanHahn changed the base branch from main to arabic-improvements June 6, 2021 18:41
@EvanHahn EvanHahn merged commit d112bfa into EvanHahn:arabic-improvements Jun 6, 2021
@EvanHahn EvanHahn mentioned this pull request Jun 6, 2021
@EvanHahn
Copy link
Owner

EvanHahn commented Jun 6, 2021

Thanks! I am going to make some changes in #199 and will merge soon.

EvanHahn pushed a commit that referenced this pull request Jun 6, 2021
Co-authored-by: Evan Hahn <[email protected]>

See [#198][0] and [#199][1].

[0]: #198
[1]: #199
@EvanHahn
Copy link
Owner

EvanHahn commented Jun 6, 2021

This has been deployed in [email protected]. Thanks again!

I would like to add you to the contributors list. What name and URL should I use?

@Misioka
Copy link
Author

Misioka commented Jun 6, 2021 via email

@EvanHahn
Copy link
Owner

EvanHahn commented Jun 6, 2021

I just credited you in 8e03ec0. Thanks again!

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