-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
[Workers] (WIP) Assets #16610
[Workers] (WIP) Assets #16610
Conversation
Deploying cloudflare-docs with Cloudflare Pages
|
Hey hey, also since I was in there, slightly updated your PR to take advantage of built-in properties ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 files reviewed, 3 outstanding issue(s) found.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5 files reviewed, 4 outstanding issue(s) found.
Hi @tanushree-sharma
I'm still learning about the product so feel free to ignore if this doesn't make sense. Thought I'd start with the higher-level suggestion, but feel free to ping me for more granular copyedits when needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
6 files reviewed, 3 outstanding issue(s) found.
Thanks @ToriLindsay for the suggestion! Getting the content up and then let's play around with structure, see what flows the best. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
10 files reviewed, 24 total issue(s) found.
Note:
This, and prior reviews, were resolved because we updated the style guide.
We'll leave a new review below.
src/content/docs/workers/configuration/versions-and-deployments/index.mdx
Outdated
Show resolved
Hide resolved
940164e
to
a4824cd
Compare
args={"--type=web-framework --experimental"} | ||
/> | ||
|
||
<DirectoryListing folder="workers/frameworks/framework-guides" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
src/content/docs/workers/static-assets/compatibility-matrix.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/workers/static-assets/compatibility-matrix.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/workers/static-assets/compatibility-matrix.mdx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple minor suggestions
7381353
to
51b1f63
Compare
* Create C3 Frameworks component and rename Typescript to TypeScript * Rename deploy-via-c3 props * Add some framework guides for Workers Assets * Update deploy command docs * Move stuff around a bit * Final touch-ups
2fbbeb6
to
7329597
Compare
0cb2d73
to
7d0c43d
Compare
* Assets docs * Frameworks Guides (#17023) * Create C3 Frameworks component and rename Typescript to TypeScript * Rename deploy-via-c3 props * Add some framework guides for Workers Assets * Update deploy command docs * Move stuff around a bit * Final touch-ups * Hide framework guides external link * Fix a bunch of dead links * compat-matrix * content updates * vitest limitation * edits from feedback * more content updates * feedback * small fix * Greg's pass of assets docs --------- Co-authored-by: Greg Brimble <gbrimble@cloudflare.com> Co-authored-by: emily-shen <69125074+emily-shen@users.noreply.github.com>
* Assets docs * Frameworks Guides (#17023) * Create C3 Frameworks component and rename Typescript to TypeScript * Rename deploy-via-c3 props * Add some framework guides for Workers Assets * Update deploy command docs * Move stuff around a bit * Final touch-ups * Hide framework guides external link * Fix a bunch of dead links * compat-matrix * content updates * vitest limitation * edits from feedback * more content updates * feedback * small fix * Greg's pass of assets docs --------- Co-authored-by: Greg Brimble <gbrimble@cloudflare.com> Co-authored-by: emily-shen <69125074+emily-shen@users.noreply.github.com>
No description provided.