Skip to content
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

chore: remove unused react imports #2937

Merged
merged 1 commit into from
May 23, 2023
Merged

chore: remove unused react imports #2937

merged 1 commit into from
May 23, 2023

Conversation

bigint
Copy link
Member

@bigint bigint commented May 23, 2023

What does this PR do?

🤖 Generated by Copilot at 1820fae

This pull request optimizes the performance of the NFT gallery component by using the memo function from 'react' to avoid unnecessary re-rendering of the DraggableCard and NftCard components. It also removes the import of React from 'react' from several files, as it is no longer needed in React 17 and above when using JSX. Additionally, it renames the file apps/web/src/components/Profile/NftGallery/Create.tsx to apps/web/src/components/Composer/ChooseThumbnail.tsx as part of a refactoring process to improve the user interface and functionality of the composer component.

Related issues

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (non-breaking small changes to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Explanation of the changes

🤖 Generated by Copilot at 1820fae

  • Removed the import of React from 'react' from several files to reduce the bundle size and improve the code quality, as it is no longer needed in React 17 and above when using JSX. (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Renamed the file apps/web/src/components/Profile/NftGallery/Create.tsx to apps/web/src/components/Composer/ChooseThumbnail.tsx, as part of a refactoring process to improve the user interface and functionality of the composer component, which allows users to create and publish posts with NFTs. (link)

Emoji

🗑️🎨🚀

@vercel
Copy link

vercel bot commented May 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
prerender ✅ Ready (Inspect) Visit Preview May 23, 2023 4:00am
web ✅ Ready (Inspect) Visit Preview May 23, 2023 4:00am

@bigint bigint enabled auto-merge (squash) May 23, 2023 03:58
@vercel vercel bot temporarily deployed to Preview – prerender May 23, 2023 04:00 Inactive
@bigint bigint merged commit 03b0873 into main May 23, 2023
6 checks passed
@bigint bigint deleted the rm-react-import branch May 23, 2023 04:05
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.

None yet

1 participant