-
Notifications
You must be signed in to change notification settings - Fork 49
Cleanup code that is leftover from the split from the Pose demo #225
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
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 cleans up code that is no longer needed after the Pose demo split and updates the README for Vercel deployment. Key changes include removal of unused imports/components, deletion of obsolete authentication-related pages, and revisions to deployment instructions.
Reviewed Changes
Copilot reviewed 80 out of 83 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
app/(dashboard)/settings/documents/page.tsx | Removed duplicate import of ConnectDocuments |
app/(dashboard)/pets/page.tsx | Removed redundant icon and Button imports |
app/(dashboard)/payouts/page.tsx | Removed unused components and streamlined imports |
app/(dashboard)/payments/page.tsx | Removed unnecessary Button and test import for payments |
app/(dashboard)/layout.tsx | Cleaned up and consolidated React and Provider imports |
app/(dashboard)/home/page.tsx | Removed unused state and constant declaration |
app/(dashboard)/finances/page.tsx | Removed unused React hooks and cleaned up unused Stripe components |
app/(dashboard)/finances/financing/page.tsx | Eliminated unnecessary state and effect hooks for clarity |
app/(auth)/signup/page.tsx | Updated import order to conform to new grouping order |
app/(auth)/register/page.tsx | Deleted obsolete registration page |
app/(auth)/register/layout.tsx | Deleted obsolete layout file for registration |
app/(auth)/onboarding/page.tsx | Reordered imports for consistency |
app/(auth)/loginas/page.tsx & form.tsx | Deleted outdated login-as related code |
app/(auth)/layout.tsx | Reordered imports for consistency |
app/(auth)/business/form.tsx | Reordered and consolidated icon imports |
README.md | Updated documentation text and instructions for deploying on Vercel |
Files not reviewed (3)
- .env.example: Language not supported
- .vscode/settings.json: Language not supported
- LICENSE: Language not supported
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.
thanks for cleaning this up -can you share more about the deployment process to vercel?
Yep, you just need to create a mongo instance somewhere. I have a deployment at mongodb.com which is actually powering https://stripe-connect-furever-demo-seven.vercel.app/ (which is a fork of this). So it's really easy and "just works" assuming you point this to a mongo instance. |
This deletes a bunch of code that was only used for the Pose site. Or, from unused imports, etc.
I also tried to update the README to make this easy to deploy to vercel too.
This also creates a dashboard link for all accounts with access to a dashboard, instead of just Express.