wepay-rails icon indicating copy to clipboard operation
wepay-rails copied to clipboard

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

Open timothyjoh opened this issue 12 years ago • 2 comments

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.

timothyjoh avatar Sep 17 '12 03:09 timothyjoh

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.

SteveAquino avatar Sep 17 '12 19:09 SteveAquino

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.

timothyjoh avatar Sep 17 '12 22:09 timothyjoh