Skip to content

docs(redis): note that TS GET parses JSON values - #1002

Merged
ytkimirti merged 1 commit into
mainfrom
DX-3078-get-json
Nov 9, 2026
Merged

ytkimirti merged 1 commit into
mainfrom
DX-3078-get-json

Conversation

@ytkimirti

@ytkimirti ytkimirti commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Preview: https://upstash-docs-git-dx-3078-get-json-c61b7f.docs7.io/redis/sdks/ts/commands/string/get

The TypeScript GET page didn't say that the SDK parses JSON values, so people called JSON.parse on an object and got an error (#425). It now says so and links to the option that returns the raw string.

Evidence: Ran against a real database with @upstash/redis 1.41.0: after set(k, { a: 1 }), get(k) returns object {"a":1}, and with automaticDeserialization: false it returns string {"a":1}.

Compatibility: None: docs-only text change.

@linear-code

linear-code Bot commented Nov 9, 2026

Copy link
Copy Markdown

DX-3078

@context7

context7 Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Docs7 for upstash.com

Result Status Action
Deployment ✅ Ready Open preview
Content review 🛠️ Found 1 problem. Fixes proposed. —

Commit 98aef0b · Updated 2026-11-09 07:28 UTC · View build details

Broken links (1)

  • redis/sdks/ts/commands/string/get.mdx:18 — Anchor disable-automatic-deserialization does not exist; heading is 'Disable automatic serialization'. 🛠️ fix proposed

Fixes opened as a pull request against this branch: #1003

@context7
context7 Bot temporarily deployed to Docs7 Preview: DX-3078-get-json November 9, 2026 07:28 Destroyed
@ytkimirti
ytkimirti marked this pull request as ready for review November 9, 2026 07:54
@ytkimirti
ytkimirti merged commit 4848574 into main Nov 9, 2026
4 checks passed
@ytkimirti
ytkimirti deleted the DX-3078-get-json branch November 9, 2026 07:54

This branch was successfully deployed

No deployments
Docs7 Preview: DX-3078-get-json — 98aef0b1 Deployed Nov 9, 2026 by context7[bot]
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.

1 participant