Skip to content

Conversation

armedi
Copy link

@armedi armedi commented Oct 8, 2025

Fixes #8059

Not only the useLayoutEffect page as reported in the above issue, the useMemo page is also affected

@Copilot Copilot AI review requested due to automatic review settings October 8, 2025 02:55
Copy link

meta-cla bot commented Oct 8, 2025

Hi @armedi!

Thank you for your pull request and welcome to our community.

Action Required

In order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you.

Process

In order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes missing file path specifications in code block declarations for React documentation examples. The changes ensure that code examples in the useLayoutEffect and useMemo documentation pages properly specify their file paths, which is necessary for the documentation's code sandbox functionality to work correctly.

  • Adds missing file path specifications to code block declarations
  • Updates four code blocks across two documentation files

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/content/reference/react/useMemo.md Adds file path specification to TodoList.js code example
src/content/reference/react/useLayoutEffect.md Adds file path specifications to three code examples (Tooltip.js and App.js)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

github-actions bot commented Oct 8, 2025

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Copy link

meta-cla bot commented Oct 8, 2025

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks!

@meta-cla meta-cla bot added the CLA Signed label Oct 8, 2025
@armedi
Copy link
Author

armedi commented Oct 8, 2025

Tagging @poteto as this PR fixes bug introduced in #8058

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: client-side exception when visiting useLayoutEffect page
1 participant