Skip to content

Conversation

@korinne
Copy link
Contributor

@korinne korinne commented Feb 3, 2025

Summary

  • Removed Workers Sites docs, since we do not recommend using Workers sites, have very little usage, and LLMs currently use our Worker Sites docs in teaching users how to use our platform.
  • Added an FAQ page under Workers Static Assets in order to explain some common questions we get asked, as well as add info about Workers Sites

Documentation checklist

  • [ X] The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2025

This PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:

  • /workers/configuration/sites/configuration/
  • /workers/configuration/sites/
  • /workers/configuration/sites/start-from-existing/
  • /workers/configuration/sites/start-from-scratch/
  • /workers/configuration/sites/start-from-worker/

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 3, 2025

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9b28e92
Status: ✅  Deploy successful!
Preview URL: https://95e9c4f2.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://korinne-workers-sites.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2025

src/env.d.ts Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this file?

@GregBrimble
Copy link
Contributor

This lil guy is right, we should set up some redirects for those pages to Workers Assets in their place.

Copy link
Contributor

@Cherry Cherry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was Workers Sites ever officially deprecated by Cloudflare?

Removing docs for a functional product that still exists within wrangler today (and hasn't yet been deprecated there so will likely still exist in v4) seems a little hasty. Pushing people to Pages and Assets with messages at the top of docs makes sense, but removing them entirely seems heavy-handed to me, even if usage is low.

LLMs pointing people to use Sites is something that should be solved without removing docs for a still-functional product, in my opinion.

@korinne
Copy link
Contributor Author

korinne commented Feb 4, 2025

@Cherry Hey! No, we haven't officially deprecated Workers Sites, though we have essentially no usage (we've been pushing people to Pages for some time, and Pages has garnered all the previous Workers Sites usage). While I'd typically agree that I don't like pulling docs down when something isn't officially deprecated, we checked our current usage and decided we were comfortable with the choice.

If anyone finds this PR and happens to absolutely need the Workers Sites docs, please feel free to reach out to me directly at [email protected], and I can help you.

Thanks for the callout @Cherry -- it's a super fair point.

@Cherry
Copy link
Contributor

Cherry commented Feb 4, 2025

Thanks @korinne, I appreciate the thoughtful response. I feel that this sets a pretty dangerous precedent though.

I myself have multiple Workers Sites projects in production, and Cloudflare's own official SDK wrangler still supports it, and doesn't warn me about my usage any time I update or deploy my sites. While the product is still actively supported there, removing docs for it is hasty, and finding a single solitary email in a GitHub issue thread for support on it is not really what I'd consider a solution unfortunately.

Ideally, wrangler should first be updated to warn of a deprecation, with these docs actively reflecting that. Then, over time, the docs tucked away, and eventually removed as the featureset gets removed from wrangler.

farshadjanu1

This comment was marked as spam.

@rickyrobinett
Copy link
Contributor

Since this product isn't deprecated yet, we shouldn't remove the docs completely. Can we instead block this page from crawlers so it's less findable (and LLMs don't crawl) but still available for folks who need it?

@kodster28
Copy link
Collaborator

Closing in favor of #19818

@kodster28 kodster28 closed this Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/l

Projects

None yet

Development

Successfully merging this pull request may close these issues.