Skip to content

Conversation

@whatsupsumit
Copy link

#3069 i have migrated it to nextjs..... css might not be that good i will enhance it if you want ...create another issue for it....
I've successfully completed the migration of the Keploy Writers Program website from static HTML/CSS/JS to Next.js 15 with TypeScript and Tailwind CSS! Here's what was accomplished:

✅ Completed Tasks
Created New Branch: nextjs-migration-3069 and pushed to GitHub
Scaffolded Next.js 15: With TypeScript, Tailwind CSS v4, and App Router
Migrated All Content: Every section from the original site
Component Architecture:
Header & Footer components
6 section components (Hero, About, Program, Process, Criteria, Community)
Reusable UI components (Button with Radix UI)
Modern Styling: Tailwind CSS with custom configuration
Image Optimization: All images moved to public and using Next.js Image
Font Optimization: Google Fonts (Lato, Inconsolata, Montserrat)
SEO: Proper metadata, OpenGraph tags, and site configuration
Build Success: ✅ Production build compiles without errors
Documentation: Created comprehensive README-NEXTJS.md, PR_DESCRIPTION.md, and MIGRATION_SUMMARY.md
🎯 Key Features
✅ Responsive design (mobile-first)
✅ Smooth scroll navigation
✅ Mobile menu with hamburger icon
✅ All CTAs and links preserved
✅ Social media integration maintained
✅ Newsletter signup functionality
✅ Custom scrollbar styling
✅ Production-ready and optimized
📦 Tech Stack

Copilot AI review requested due to automatic review settings October 7, 2025 10:22
Copy link

Copilot AI left a 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 migrates the Keploy Writers Program website from static HTML/CSS/JS to Next.js 15 with TypeScript and Tailwind CSS. The migration maintains all existing content and design while implementing modern web development practices.

  • Complete architecture overhaul to Next.js 15 with App Router and TypeScript
  • Component-based structure with proper separation of concerns
  • Modern styling using Tailwind CSS v4 and responsive design patterns

Reviewed Changes

Copilot reviewed 29 out of 106 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.json Defines Next.js project dependencies and scripts
next.config.ts Next.js configuration with image optimization and build settings
tailwind.config.ts Tailwind CSS configuration with custom fonts and theme
app/layout.tsx Root layout with metadata, fonts, and component structure
app/page.tsx Main page composing all sections
components/sections/* Individual page sections converted to React components
components/Header.tsx Responsive header with mobile navigation
components/Footer.tsx Footer with newsletter and social links
components/ui/button.tsx Reusable button component with variants
public/css-legacy/* Legacy CSS files preserved for reference

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you and congratulations 🎉 for opening your very first pull request in writers-program

Srinu346 and others added 6 commits October 7, 2025 16:12
* Updated New Icons and CDN link of font awesome

Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>

* fix: update old icons

Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>

---------

Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
…#3069

Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
Signed-off-by: Mr Sumo <sksumitboss123@gmail.com>
@whatsupsumit whatsupsumit force-pushed the nextjs-migration-3069 branch from 73d04bf to d0c4812 Compare October 7, 2025 10:48
@amaan-bhati
Copy link
Member

Hey @whatsupsumit thanks for raising this pr, can you please share a preview video/screenshot before i review this on my local? Also, can you please confirm if nothing fails on running npm run build

@whatsupsumit
Copy link
Author

whatsupsumit commented Oct 7, 2025

Screenshot 2025-10-07 163801 Screenshot 20@amaan-bhati 5-10-07 163822 image

@amaan-bhati can you please add hacktoberfest labels....
i can enhance the frontend ....
if you want me ...create a diff issue for it..🫡😅

@amaan-bhati
Copy link
Member

amaan-bhati commented Oct 7, 2025

Hey @whatsupsumit sure, i will add the label here but before i actually try and merge this, the ui pr needs to address all the expected outputs, the design currently looks broken and not identical to what we currently have. Even if you are not making it completely identical, we atleast need a standing frontend that does not break and follow the best practices in the code. I will create a different issue for the ui enhancement but even if we merge this pr, we need a standing frontend that doesnt break.

@whatsupsumit
Copy link
Author

@amaan-bhati ok gotcha....i am working on it...😅🫡

@whatsupsumit
Copy link
Author

@amaan-bhati bro i am getting a irritating bug .... i am not able to resolve it....i will try to raise PR by today...If i didnt do it today....
😅then i died trying...😅😅

@amaan-bhati
Copy link
Member

Hey @whatsupsumit Can you please confirm if you are still working on this, is there an update on the same?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants