[joysl] String replace needs a string attribute

awslabs / aws-iot-greengrass-accelerators

Deployable solutions for AWS IoT Greengrass for common design patterns, or to address specific use-cases.

MIT No Attribution

70 stars 33 forks source link

[joysl] String replace needs a string attribute #28

Closed joysl closed 1 year ago

joysl commented 1 year ago

Issue #, if available:

Description of changes: While processing a JSON ggv2 recipe the component function will throw an error if the value of a JSON element is a number. Since replace method only applies to str

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

gadams999 commented 1 year ago

Thanks for the PR. I'll run through the change and get it merged in. About time for unit tests I reckon.