fix: cloudflare_workers_script run_worker_first accepts list input

[maxwellpeterson]

• [x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

• Update assets.config.run_worker_first property of cloudflare_workers_script resource to accept list of strings in addition to boolean input
• The Cloudflare API accepts both types of values, so this resource should support both

Additional context & links

Fixes #5956

[1000hz]

@aashishshrestha09 the bug you ran into was due to a mismatch between the attribute type within the model and schema. Sorry for the trouble here. This PR handles both boolean and string[] cases by using a Dynamic attribute capable of accepting both types.

[aashishshrestha09]

@aashishshrestha09 the bug you ran into was due to a mismatch between the attribute type within the model and schema. Sorry for the trouble here. This PR handles both boolean and string[] cases by using a Dynamic attribute capable of accepting both types.

Hi @1000hz,

Thank you for clarifying! That makes sense. Once this PR is merged, will the fix be available immediately in a v5.10.1 patch release, or will we need to wait for the next provider release (v5.11.0) to use it in Terraform?

Thanks again! Also, I’ve detected another issue with the assets block and have created a bug report here.

[vaishakdinesh]

Can you target next please? And make sure its branched off next too.

[vaishakdinesh]

lgtm

[vaishakdinesh]

@maxwellpeterson Do we still need this? I think these changes are already in?

[mgirouard]

Hey @maxwellpeterson I rebased your PR and it looks like there's no diff. I think other patches may have take care of this already maybe?