Change type for Calc Fields from BigDecimal to String#34
Conversation
…here is no error when the data represents a Date, DateTime, or Time
b064c1e to
c3c456e
Compare
|
@AaronJRubin |
|
@tomohiro-okuyama Have you been able to find a good solution to mitigate this issue that would not introduce a breaking change? |
|
@AaronJRubin We are going to merge this PR soon first. After that, we'll add above changes and release it as v1.4.0. |
|
I see - that sounds like a great solution! Thanks so much for moving this forward. |
|
@AaronJRubin |
As documented at https://developer.cybozu.io/hc/ja/articles/202166330-%E3%83%95%E3%82%A3%E3%83%BC%E3%83%AB%E3%83%89%E5%BD%A2%E5%BC%8F, a Calc field can contain a string representation of a Date, DateTime, or Time in addition to a number. Therefore, we cannot assume that the data will be parsable as a BigDecimal, and the simplest/cleanest fix is just to keep the value as a string and let the consumer of the API handle appropriately.