Skip to content

Conversation

@yuechao-qin
Copy link
Collaborator

@yuechao-qin yuechao-qin commented Jan 16, 2026

Description

Added PublishComponent.tsx to the React Compiler enabled directories and simplified the component by replacing useCallback with a regular function for the onChange handler.

Type of Change

  • Improvement
  • Cleanup/Refactor

Tests

  • Confirm Chrome React Extension shows Memo next to PublishComponent.
    • @Mbeaulne I wasn't able to search PublishComponent, but I believe the Anonymous is memoized which appears to be the component itself?
    • image.png
  • Ran npm run validate:test and passed.

Copy link
Collaborator Author

yuechao-qin commented Jan 16, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@yuechao-qin yuechao-qin marked this pull request as ready for review January 16, 2026 19:04
@yuechao-qin yuechao-qin requested a review from a team as a code owner January 16, 2026 19:04
@graphite-app graphite-app bot changed the base branch from ycq/manage-component-publish-button to graphite-base/1653 January 19, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants