Skip to content

Refactor ToOrdinalWords to use existing NumberToWordsExtension#186

Merged
MehdiK merged 3 commits intoHumanizr:masterfrom
thunsaker:refactor-ToOrdinalWords
Apr 11, 2014
Merged

Refactor ToOrdinalWords to use existing NumberToWordsExtension#186
MehdiK merged 3 commits intoHumanizr:masterfrom
thunsaker:refactor-ToOrdinalWords

Conversation

@thunsaker
Copy link
Contributor

Refactored ToOrdinalWords to use existing NumberToWordsExtension instead of a separate NumberToOrdinalWordsExtension extension.

This change allows contributors to start localizing the Ordinal words.

release_notes.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

You've added your PR to v1.20.2. Please add it to In Development section.

@thunsaker
Copy link
Contributor Author

Fixed!

MehdiK added a commit that referenced this pull request Apr 11, 2014
Refactor ToOrdinalWords to use existing NumberToWordsExtension
@MehdiK MehdiK merged commit 3c07ac9 into Humanizr:master Apr 11, 2014
@MehdiK
Copy link
Collaborator

MehdiK commented Apr 11, 2014

That was quick :) Thanks.

@thunsaker thunsaker deleted the refactor-ToOrdinalWords branch April 11, 2014 18:12
@thunsaker
Copy link
Contributor Author

The web UI is very useful for these situations.

Now I can work on my Spanish ToOrdinalWords PR later today/tomorrow.

@MehdiK
Copy link
Collaborator

MehdiK commented Apr 11, 2014

Cool :) Looking forward to it. That would be the first localisation of ordinal words.

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