Skip to content

docs(cacheTag): remove unused cacheLife import and add missing switcher#72217

Merged
devjiwonchoi merged 1 commit into
vercel:canaryfrom
Juneezee:docs/cacheTag
Nov 3, 2024
Merged

docs(cacheTag): remove unused cacheLife import and add missing switcher#72217
devjiwonchoi merged 1 commit into
vercel:canaryfrom
Juneezee:docs/cacheTag

Conversation

@Juneezee
Copy link
Copy Markdown
Contributor

@Juneezee Juneezee commented Nov 2, 2024

This PR fixes two issues on the cacheTag documentation page:

  1. Remove the unused unstable_cacheLife import in the TypeScript codeblock of Creating tags from external data section:

    image

  2. Missing switcher for TS codeblock in Creating tags from external data and Invalidating tagged cache sections. We already have the corresponding JS codeblock in the Markdown.

    image

    image

…tcher`

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Nov 2, 2024
@ijjk
Copy link
Copy Markdown
Member

ijjk commented Nov 2, 2024

Allow CI Workflow Run

  • approve CI run for commit: c8c1559

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

1 similar comment
@ijjk
Copy link
Copy Markdown
Member

ijjk commented Nov 2, 2024

Allow CI Workflow Run

  • approve CI run for commit: c8c1559

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@devjiwonchoi
Copy link
Copy Markdown
Member

Thank you for the PR!

@Juneezee
Copy link
Copy Markdown
Contributor Author

Juneezee commented Nov 3, 2024

@devjiwonchoi Thanks for reviewing the PR! If you have a few spare minutes, would you like to take a look at the other two similar PRs #72215 and #72216 as well?

@devjiwonchoi devjiwonchoi merged commit 0c14b38 into vercel:canary Nov 3, 2024
@devjiwonchoi
Copy link
Copy Markdown
Member

Sure! Thank you for your contribution!

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Documentation Related to Next.js' official documentation. locked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants