STRATCONN-6101/wrong-description

[arnav777dev]

Proposed changes

A customer reached out letting us know that the Kinesis Firehose destination is missing a description for the mixed setting. Raised the question in the #questions-destinations Slack channel

Added empty array check to defaultValue condition in destination footer

• Add check for '[]' to prevent displaying "defaults to" text for empty arrays

Merge timing

• ASAP once approved

Closes #5673

[forstisabella]

HI @arnav777dev - when I build the site locally it's still broken.

[arnav777dev]

Hey @stayseesong,
I added a quick debug line to inspect the defaultValue:
<p style="color:red;">DEBUG: '{{ item.defaultValue }}' (type: {{ item.type }})</p>
As seen in the attached screenshot, it appears to be empty ""— but the condition
{% if item.defaultValue != null and item.defaultValue != "" and item.defaultValue != '{}' and item.defaultValue != '[]'%}
still evaluates as true, leading to an output like defaults to ., which might not be ideal.

I’m not too familiar with Liquid quirks here — maybe someone closer to the docs setup might spot what’s going on.
cc - @forstisabella
Thanks!

[forstisabella]

hi @arnav777dev - I'll take a look and see if I can figure out what's going on. In the meantime: the Slack thread mentioned this might need to be updated to be a required setting. Are you able to do that on your end?

[forstisabella]

hi @arnav777dev - see the screenshot below, I think we fixed it! 🥳

[stayseesong]

@forstisabella hmm am I doing something wrong? When I run it locally, I don't see the fix