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

chore(examples): updated the example of with-styled-components-rtl to utlize the App router #74180

Merged

Conversation

Manoj-M-S
Copy link
Contributor

This PR updates the with-styled-components-rtl example to use the App Router. Here are the changes that have been made:

  • Renamed the pages folder to the app folder.
  • Converted the index.tsx to page.tsx file as part of the App Router.
  • Created layout.tsx in app directory, merging functionality from both _app.tsx and _document.tsx:
    • ThemeProvider setup from _app.tsx
    • RTL and styled-components setup from _document.tsx
  • Created lib/registry.tsx for styled-components server-side rendering and moved styled-components server-side rendering logic from _document.tsx
  • Updated the package.json file.

The following actions were performed as part of this PR:

  • Ran pnpm prettier-check with no issues found.
  • Executed the pnpm check-examples script.

CC: @samcx

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

ijjk commented Dec 20, 2024

Allow CI Workflow Run

  • approve CI run for commit: 6e20981

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

@ijjk
Copy link
Member

ijjk commented Dec 20, 2024

Allow CI Workflow Run

  • approve CI run for commit: db5e32c

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 January 7, 2025 20:23
@samcx
Copy link
Member

samcx commented Jan 7, 2025

@Manoj-M-S We should remove the example as the dependency has not been updated in a while.

@samcx samcx enabled auto-merge (squash) January 7, 2025 20:23
@samcx samcx merged commit 6d69257 into vercel:canary Jan 7, 2025
39 checks passed
wbinnssmith pushed a commit that referenced this pull request Jan 10, 2025
… utlize the App router (#74180)

This PR updates the with-styled-components-rtl example to use the App
Router. Here are the changes that have been made:

- Renamed the `pages` folder to the `app` folder.
- Converted the `index.tsx` to `page.tsx` file as part of the App
Router.
- Created `layout.tsx` in app directory, merging functionality from both
` _app.tsx` and `_document.tsx`:
  -  `ThemeProvider` setup from `_app.tsx`
  -   `RTL` and `styled-components` setup from `_document.tsx`
- Created `lib/registry.tsx` for styled-components server-side rendering
and moved styled-components server-side rendering logic from
`_document.tsx`
- Updated the package.json file.

The following actions were performed as part of this PR:

- Ran `pnpm prettier-check` with no issues found.
- Executed the `pnpm check-examples` script.

CC: @samcx

---------

Co-authored-by: samcx <sam@vercel.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 22, 2025
# 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