Add BigDecimal inline array property type
ljacqu opened this issue · 8 comments
Can only be done when #230 is closed: introduce an inline array property type constant for the type BigDecimal. It should behave like the other number types.
To do
- Add a BIG_DECIMAL constant for the BigDecimal number type (between Double and String)
- Adapt tests
hey!
i would like to contribute on this issue can you assign me to explore more in it.🙂
Will do! Just please note this cannot be started before the linked issue is done. I will comment again when this can be picked up (should be in 1-2 days)
Thank you for letting me know. I completely understand, and I'm happy to wait until the linked issue is resolved. Please take your time to address it, and once it's done, I'll be ready to pick up this task. Feel free to reach out when you're ready for me to proceed, and I'll be happy to contribute. Looking forward to it!
hi @Akshayp02, thanks for your patience! The relevant code changes are now available on master, meaning that this is not blocked anymore. The starting point for this issue is InlineArrayPropertyType.
Hello Mam
I hope you're doing well. I'm currently working on the project and have some questions about a specific task. I'm a bit confused about which file I should be working on for this task. Could you please provide a brief description or guidance on where I should focus my efforts?
Thank you for your assistance.
Hi, essentially the InlineArrayPropertyType offers a property type for various types, but it's missing an implementation for BigDecimal. So the same way there is an inline array property type for double or String, we'd want one for the BigDecimal type there. This task entails adding that and extending the tests in InlineArrayPropertyTypeTest so that the new BigDecimal type is also covered.