Skip to content
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 method parameter for requestInit with rescript v11 #129

Merged
merged 3 commits into from
Jan 16, 2024

Conversation

WhyThat
Copy link
Contributor

@WhyThat WhyThat commented Jan 16, 2024

Hello, thanks for this package :)

Using @as directive as no impact for rescript v9 or v10 but it works in rescript v11

with rescript v9
image
With rescript v10
image
With rescript v11
image

fix #128

@TheSpyder
Copy link
Owner

TheSpyder commented Jan 16, 2024

Oh that's much better than how I was going to do it (I thought duplicate bindings would be required). Thanks!

We will eventually remove the _ (see #120) but this is a nice change that can be deployed safely.

@TheSpyder TheSpyder merged commit 44cc088 into TheSpyder:main Jan 16, 2024
@TheSpyder
Copy link
Owner

Published as 0.9.1 👍

@WhyThat
Copy link
Contributor Author

WhyThat commented Jan 17, 2024

Thanks for the quick release :)

@WhyThat WhyThat deleted the fix-fetch-method branch January 17, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fetch RequestInit method parameter issue
2 participants