Skip to content

Conversation

abdallah-abdelfatah
Copy link

I was recently working on some changes on maple/horizontal FE changes, and I notice that we're using our fork of humanize duration. But it's lacking the proper plural rules for AR as per language plural rules, I can also confirm that's correct since it's my own language. : ))

upon further investigation, it turns out that this change was done to original repo, check getArabicForm function here.

This pr is cpying the minimal changes for the proper plural form in AR, it's missing some stuff of things (tests) but I went with this quick and dirty way to see if I should but more time in this.

This is currently in my own fork, I'll either keep it or move depending on the maintainer decision.

So let me know,

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.

1 participant