-
Notifications
You must be signed in to change notification settings - Fork 0
feat(web-server): set up web server with react using vite #2
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
base: master
Are you sure you want to change the base?
Conversation
ziiqii
commented
Sep 19, 2025
- Set up web app with React using Vite
- Created landing, home, profile pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR sets up a React web application using Vite as the build tool and Material-UI for the component library. It creates a basic PeerPrep application with authentication flow and user navigation.
- Created foundational React app structure with TypeScript configuration
- Implemented three main pages: landing page for authentication, home page for settings selection, and profile page for user management
- Set up routing with React Router and a sidebar layout for authenticated pages
Reviewed Changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
vite.config.ts | Basic Vite configuration with React plugin |
tsconfig.json | Main TypeScript configuration for the React app |
tsconfig.node.json | TypeScript configuration for build tooling |
package.json | Project dependencies and scripts setup |
index.html | HTML entry point for the React application |
src/main.tsx | Application entry point with routing configuration |
src/theme.tsx | Material-UI dark theme configuration |
src/components/Layout.tsx | Main layout component with sidebar navigation |
src/pages/Landing.tsx | Landing page for user authentication |
src/pages/Home.tsx | Home page with difficulty and language selection |
src/pages/Profile.tsx | User profile management page |
.gitignore | Git ignore rules for build outputs and dependencies |
Files not reviewed (1)
- web-server/package-lock.json: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
b3824e6
to
b0ed0ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- web-server/package-lock.json: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
||
const handleSave = () => { | ||
// Save profile logic here | ||
alert('Profile saved!'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using alert()
is not a good user experience practice. Consider replacing with a toast notification or snackbar component from Material-UI for better UX.
Copilot uses AI. Check for mistakes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM