decimalfield should specify step=any

[jazzband]

decimalfield should specify step=any

[caacree]

Not sure why this hasn't been implemented, I just ran into it as well. It's an easy fix, but should definitely be implemented into the field.

Instead of step="any", set it to be based on the number of decimal places. ie. step=".01" for decimal places=2 on the field, etc.

[gregmuellegger]

Yep that's a great idea! I think that ticket just slipped through our attention.
I would be great to have this in the project. Would you like to implement that in a pull request?

[caacree]

I've never contributed to a project before, but I'll give it a shot. Give me a few hours and I'll be back to work it out.

[gregmuellegger]

Great that you want to give it a shot! Feel free to ask any question if you can't get along.
A good place to start is usually to write a failing test. Because then you have the confidence that the patch you are developing will do exactly what you want.

[marcoala]

Stepped into this one as well, @caacree are you still working on this? I could try to write a patch for it

[gregmuellegger]

@marcoala Sure, go ahead :) Cheers for stepping up.

[caacree]

Sorry guys, totally forgot about this until I saw it in my email this morning. Honestly slipped my mind.

Just submitted the pull request.

[caacree]

Hey Greg, sorry about that the first time around. Let me know if the tests pass now (not sure if there's a way I can see TravisCI stuff inside Github or I need it installed myself?).

Thanks.

[gregmuellegger]

I just release django-floppyforms 1.5.0 which includes the changes for this ticket. Happy coding.