Deleting get-started page#1449
Conversation
|
@steam-bell-92 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
There was a problem hiding this comment.
Pull request overview
Removes the Docusaurus /get-started page from src/pages, eliminating its React implementation and associated CSS module.
Changes:
- Deleted
src/pages/get-started/index.tsx(page implementation). - Deleted
src/pages/get-started/styles.module.css(page styling).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/pages/get-started/index.tsx | Removes the /get-started route by deleting the page component. |
| src/pages/get-started/styles.module.css | Removes the styling that was only used by the deleted page. |
Comments suppressed due to low confidence (1)
src/pages/get-started/index.tsx:1
- This PR deletes the
/get-startedpage route, but there are still multiple in-repo links pointing to/get-started//get-started/(e.g., header CTA, footer links, interview-prep CTAs). After this change those links will 404 in production. Either update those links to the new intended destination (docs/community/etc.) or add a redirect for/get-startedto the replacement page before removing this route.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
I’ll redirect all the other links in the codebase that were somehow connected to the get-started page to more relevant links or repos after this PR is merged. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
im putting this on hold there is something broken internally, vercel keep failing to deploy |
|
@sanjay-kv i think there is something wrong with the vercel ATM |
Description
Fixes # (issue)
Type of Change
Changes Made
Dependencies
Checklist
npm run buildand attached screenshot(s) in this PR.