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

Ordinalize not working in PT, only for PT-BR #415

Closed
tlfbrito opened this issue May 13, 2015 · 6 comments
Closed

Ordinalize not working in PT, only for PT-BR #415

tlfbrito opened this issue May 13, 2015 · 6 comments

Comments

@tlfbrito
Copy link

I'm trying to use the Ordinalize to transform 1 into 1st with
1.Ordinalize() but the result was 1 again.

Do I need to to a special configuration?
I also have tried other example: "men".ToQuantity(2) => "2 men" and it worked...

@tlfbrito
Copy link
Author

This is related to #416 because my culture is defined as pt and not pt-br.
I've created a PR to fix this issue.

@tlfbrito
Copy link
Author

@MehdiK Is there a way to register new Ordinalizers from custom classes? If yes, can you post an example?
Thanks in advance!

@tlfbrito tlfbrito changed the title Ordinalize not working Ordinalize not working in PT, only for PT-BR May 14, 2015
@MehdiK
Copy link
Member

MehdiK commented May 15, 2015

@TiagoBrito thanks for reporting the error and submitting the PR.

Is there a way to register new Ordinalizers from custom classes? If yes, can you post an example?

You should be able to do it by Configurator.Ordinalizers.Register("pt", new PortugueseOrdinalizer()).

@tlfbrito
Copy link
Author

If you need more work in PT/PT-BR let me know

@clairernovotny
Copy link
Member

Was the corresponding PR merged? Is this still an open issue?

@SimonCropp
Copy link
Collaborator

Closing due to no response

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

No branches or pull requests

4 participants