Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Migrated svg-components to App-router #68394

Merged
merged 5 commits into from
Aug 2, 2024

Conversation

Hemanshu-Upadhyay
Copy link
Contributor

Hello! This PR Contains the Following Changes

=> svg-components Example Migrated to App-Router
=> Added the layout.tsx file as part of the App Router.
=> Folder Structure Updated to support APP-Router convention
=> File Convention updated to Support the APP-Router convention
=> Npm package got updated "@svgr/webpack": "^6.2.1", to "@svgr/webpack": "^6.5.1",

CC- @samcx

@ijjk ijjk added the examples Issue was opened via the examples template. label Aug 1, 2024
@ijjk
Copy link
Member

ijjk commented Aug 1, 2024

Allow CI Workflow Run

  • approve CI run for commit: d8e1b7c

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

@samcx samcx self-requested a review August 1, 2024 19:26
@samcx
Copy link
Member

samcx commented Aug 1, 2024

@Hemanshu-Upadhyay Have you tried upgrading all the packages to the latest (e.g., https://github.com/gregberge/svgr/releases/tag/v8.1.0)?

@Hemanshu-Upadhyay
Copy link
Contributor Author

Updated All the Packages
CC - @samcx

Copy link
Member

@samcx samcx left a comment

Choose a reason for hiding this comment

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

Thank you for submitting a PR!

:lfgtm

@samcx samcx merged commit 599f3a0 into vercel:canary Aug 2, 2024
38 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
examples Issue was opened via the examples template. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants