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

[examples] Add Joy UI + Vite.js + TypeScript example app #37406

Merged
merged 7 commits into from
Aug 9, 2023

Conversation

nithins1
Copy link
Contributor

Helps address problems described in #37367 (too many CRA examples, not enough Vite)

@mui-bot
Copy link

mui-bot commented May 26, 2023

Netlify deploy preview

https://deploy-preview-37406--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 8e69be5

@zannager zannager added the examples Relating to /examples label May 26, 2023
@zannager zannager requested a review from mnajdova May 26, 2023 08:10
@mnajdova mnajdova requested review from siriwatknp and samuelsycamore and removed request for mnajdova May 31, 2023 11:33
@nithins1 nithins1 requested a review from siriwatknp June 2, 2023 20:28
@ZeeshanTamboli ZeeshanTamboli changed the title [examples] Joy + Vite.js + TypeScript example app [examples] Add Joy + Vite.js + TypeScript example app Aug 9, 2023
@ZeeshanTamboli ZeeshanTamboli changed the title [examples] Add Joy + Vite.js + TypeScript example app [examples] Add Joy UI + Vite.js + TypeScript example app Aug 9, 2023
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

@nithins1 Thanks for the PR and contribution! I made a few changes and it looks good to go.

@ZeeshanTamboli ZeeshanTamboli merged commit 5f9f8d6 into mui:master Aug 9, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Relating to /examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants