get started button redirection#1436
Closed
steam-bell-92 wants to merge 4 commits intorecodehive:mainfrom
Closed
Conversation
…-started page from codebase
Contributor
|
@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. |
github-actions
Bot
added
in-review
|
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! 😊 |
|
✅ Synchronized metadata from Issue #1419:
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the custom /get-started page and updates the primary “Get Started” CTA to send users to the documentation section instead.
Changes:
- Deleted the
/get-startedpage implementation and its CSS module. - Updated the homepage header “Get Started” button link target to the docs.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/pages/get-started/index.tsx |
Removed the /get-started page implementation. |
src/pages/get-started/styles.module.css |
Removed styling specific to the deleted /get-started page. |
src/components/header/header.tsx |
Updated the “Get Started” CTA link target to docs. |
Comments suppressed due to low confidence (1)
src/components/header/header.tsx:66
- The header CTA now hardcodes an absolute docs URL via
href="https://www.recodehive.com/docs/". Since this is an internal route (docs are served at/docsin docusaurus.config.ts), this will force a full page load and break local/staging deployments that aren’t onwww.recodehive.com. Prefer an internal navigation (to="/docs"orto="/docs/").
Also, the /get-started page is deleted in this PR but there are still in-app links pointing to /get-started (e.g., src/theme/Footer/Layout/index.tsx and src/pages/interview-prep/*). Either update those links to the new docs destination or add a redirect (e.g., Docusaurus client redirects) so users don’t hit 404s.
<Link
href="https://www.recodehive.com/docs/"
className="chh__header-content__input--button"
>
Get Started
</Link>
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
✅ Synchronized metadata from Issue #1419:
|
Member
Author
|
Now this PR also fixes #1413 |
sanjay-kv
approved these changes
May 6, 2026
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
|
This has created so many broken links which cause vercel to fail |
Member
Author
|
@sanjay-kv this PR also had changes of issue #1419 so i was thinking of deleting this current PR raise another one for #1419 then delete get-started page in 2nd PR then redirection in 3rd PR. 2nd n 3rd PR will be for issue #1413 May i proceed or wait for sometime |
Member
Go ahead @steam-bell-92 . closing this PR as per now |
Member
Author
|
Even tho this PR was closed it still shows in open |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR redirects get started button to docs and deletes get-started page from codebase
Fixes #1419
Type of Change
Changes Made
Checklist
npm run buildand attached screenshot(s) in this PR.