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

Also noticed a problem with precision and scale in the decimal columns #24

Open
timothyjoh opened this issue Sep 17, 2012 · 2 comments
Open
Assignees

Comments

@timothyjoh
Copy link

In the WePayCheckoutRecords migration, there is no definition to the precision and scale attributes on the :decimal columns, so Rails assumes a precision of 10 and a scale of 0, truncating any cents returned from Wepay.

I would suggest you put in a precision of 10, and scale of 2 in those columns.

@SteveAquino
Copy link
Collaborator

Thanks for pointing that out! I've only been using whole numbers up until now which is why I didn't see this sooner, will get this fixed in the next update.

@ghost ghost assigned SteveAquino Sep 17, 2012
@timothyjoh
Copy link
Author

Yea I didn't feel it was worth a pull request.

Timothy Johnson
www.wearefound.com
540 908-4445

http://youtube.com/foundvideostudio

http://youtube.com/jhnsntmthy

On Sep 17, 2012, at 3:27 PM, Steve Aquino [email protected] wrote:

Thanks for pointing that out! I've only been using whole numbers up until now which is why I didn't see this sooner, will get this fixed in the next update.


Reply to this email directly or view it on GitHub.

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

2 participants