-
Notifications
You must be signed in to change notification settings - Fork 636
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(cloudflare_worker): add missing js module option to fix upload #1865
fix(cloudflare_worker): add missing js module option to fix upload #1865
Conversation
changelog detected ✅ |
b003edc
to
53b2e1d
Compare
53b2e1d
to
942ed14
Compare
can we please add some acceptance tests to cover this updated behaviour? i'd like to make sure we have the coverage in place for this given it required a few updates to the core request/response lifecycle. |
d3d1d1d
to
07a988c
Compare
i've fixed the docs + added a test for this which is now all passing
|
thanks for this one @js0cha 👏 |
This functionality has been released in v3.23.0 of the Terraform Cloudflare Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
Closes #1417