这是indexloc提供的服务，不要输入任何密码
Skip to content

fix(client-fetch): serialize body with falsey but valid json values #2590

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

fix(client-fetch): serialize body with falsey but valid json values #2590

wants to merge 2 commits into from

Conversation

@LinuCC
Copy link
Contributor

@LinuCC LinuCC commented Sep 4, 2025 •
edited
Loading

Fixes #2589 .

Edit: Hmm, this looks to also be fixed in #2564 for every client, might make sense to go with that PR and close this one.

@bolt-new-by-stackblitz
Copy link

bolt-new-by-stackblitz bot commented Sep 4, 2025

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@vercel
Copy link

vercel bot commented Sep 4, 2025

@LinuCC is attempting to deploy a commit to the Hey API Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link

changeset-bot bot commented Sep 4, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: e17f7e8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 4, 2025
@LinuCC LinuCC mentioned this pull request Sep 4, 2025
Closed
@LinuCC LinuCC force-pushed the client-fetch-json-serialize-valid-falsey-values branch from f480543 to 8261aba Compare September 4, 2025 06:52
@dosubot dosubot bot added bug 🔥 Something isn't working client Client package related labels Sep 4, 2025
@LinuCC LinuCC force-pushed the client-fetch-json-serialize-valid-falsey-values branch 2 times, most recently from cf74181 to ffb35f2 Compare September 4, 2025 06:55
@LinuCC LinuCC force-pushed the client-fetch-json-serialize-valid-falsey-values branch from ffb35f2 to ca81842 Compare September 4, 2025 06:58
@mrlubos mrlubos closed this Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug 🔥 Something isn't working client Client package related size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

client-fetch does not body-serialize valid json values that are falsey

2 participants

@LinuCC @mrlubos