Skip to content

Conversation

@shrink
Copy link
Contributor

@shrink shrink commented Apr 24, 2025

Summary

I was browsing the kv documentation and encountered some 404s when clicking through to the API documentation. The problem looks to be caused by the introduction of nesting in the kv API documentation, e.g:

- /api/resources/kv/subresources/namespaces/methods/bulk_delete/
+ /api/resources/kv/subresources/namespaces/subresources/keys/methods/bulk_delete/

While making this PR, I found broken links in the kv release notes, so I fixed them... but not sure if editing release notes after the fact is bad. I can remove that change if it's not good.

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Apr 24, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Updated KV API documentation to correct broken links caused by API nesting changes and fixed additional broken links in release notes.

  • Corrected API URLs in multiple documentation files to reflect new nesting structure.
  • Fixed broken links in the KV release notes.
  • Updated references to the REST API for bulk operations.

Modified Files

  • src/content/docs/kv/api/delete-key-value-pairs.mdx
  • src/content/docs/kv/api/write-key-value-pairs.mdx
  • src/content/docs/kv/index.mdx
  • src/content/docs/kv/platform/pricing.mdx
  • src/content/docs/kv/reference/faq.mdx
  • src/content/docs/kv/workers-kv-api.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@Oxyjun
Copy link
Contributor

Oxyjun commented May 20, 2025

@shrink Thank you so much for catching these links!

I've approved + enabled auto-merge.

@Oxyjun Oxyjun enabled auto-merge (squash) May 20, 2025 08:09
@Oxyjun Oxyjun merged commit 23c4995 into cloudflare:production May 20, 2025
8 checks passed
@workers-devprod workers-devprod added the contribution [Holopin] Recognizes a docs contribution, big or small label May 20, 2025
@holopin-bot
Copy link

holopin-bot bot commented May 20, 2025

Congratulations @shrink, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cmaw8uj2q12450dl7ct6ig2hh

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

sdnts pushed a commit to sdnts/cloudflare-docs that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution [Holopin] Recognizes a docs contribution, big or small product:kv size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants