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

or-OdiScript input method. #403

Merged
merged 3 commits into from
Dec 22, 2015
Merged

or-OdiScript input method. #403

merged 3 commits into from
Dec 22, 2015

Conversation

manojsahukar
Copy link
Contributor

Hello! Requesting to add this input method to the 'or' rules.
Community discussion and consensus: https://goo.gl/TbonxO and http://goo.gl/l39acO
Also, please refer to the following discussion: #402

Thank you.

@psubhashish
Copy link
Contributor

Can anyone please take a look at this and add this?

@kartikm
Copy link
Member

kartikm commented Dec 8, 2015

There are small issues. Please fix it and it should be ready:

[L19:C37] W044: Bad or unnecessary escaping.
['([କ-ହ]୍[କ-ହ])@@', 'ର୍$1'], //reph

[L20:C31] W044: Bad or unnecessary escaping.
['([କ-ହ])@@', 'ର୍$1'], //reph

[L25:C17] W044: Bad or unnecessary escaping.
['୍\ ', '୍ '], // ending with halant+zwnj

[L38:C32] W044: Bad or unnecessary escaping.
['([କନପସମ])Q', 'ତ୍$1' ],

@kartikm
Copy link
Member

kartikm commented Dec 11, 2015

@psubhashish Can you fix above error and resubmit pull request?

@psubhashish
Copy link
Contributor

Can you remove these four and submit. Manoj is not available at this moment and there are more changes to be made soon by another contributor as he has found out many new needs.

@kartikm
Copy link
Member

kartikm commented Dec 11, 2015

I'm not sure I can edit other's pull request.

psubhashish added a commit to psubhashish/jquery.ime that referenced this pull request Dec 11, 2015
As wikimedia#403 was not getting pushed and this is discussed at https://phabricator.wikimedia.org/T120634.
amire80 added a commit that referenced this pull request Dec 22, 2015
@amire80 amire80 merged commit ea97e4b into wikimedia:master Dec 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants