Feat/redesign programs page#793
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
A redesign of the Programs page to improve modularity and visual appeal by introducing reusable components and restructuring layout
- Added
ProgramCard,Hero, andCTAcomponents for consistent styling and reuse - Refactored
Programspage to useHeroAreafor the header andProgramCardfor listing programs - Updated page title, SEO metadata, and introductory text
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/pages/programs.tsx | Refactored to include HeroArea, program cards grid, updated title and layout |
| src/components/program-card/index.tsx | Added ProgramCard component for individual program display |
| src/components/hero/index.tsx | Introduced Hero component for header section |
| src/components/cta/index.tsx | Created CTA component for call-to-action sections |
Comments suppressed due to low confidence (1)
src/components/program-card/index.tsx:13
- [nitpick] There are currently no tests covering the
ProgramCardcomponent; adding unit or snapshot tests would help ensure it renders correctly and handles props as expected.
const ProgramCard = ({ program }: Props) => {
| return ( | ||
| <div className="tw-bg-[#091f40] tw-text-white tw-py-20"> | ||
| <div className="tw-container"> | ||
| <h1 className="tw-text-4xl tw-font-bold">Our Program</h1> |
There was a problem hiding this comment.
The heading text reads “Our Program” (singular), but should likely be “Our Programs” to align with the page’s plural context.
Suggested change
| <h1 className="tw-text-4xl tw-font-bold">Our Program</h1> | |
| <h1 className="tw-text-4xl tw-font-bold">Our Programs</h1> |
| ))} | ||
| </div> | ||
| </section> | ||
| <hr className="tw-my-12 tw-border-t tw-border-gray-200" /> |
There was a problem hiding this comment.
The CTA component is defined but not used on this page; consider importing and placing <CTA /> after the horizontal rule to prompt users to apply.
Suggested change
| <hr className="tw-my-12 tw-border-t tw-border-gray-200" /> | |
| <hr className="tw-my-12 tw-border-t tw-border-gray-200" /> | |
| <CTA /> |
|
🔍 Code Quality Score Breakdown:
💡 Recommendations:
|
jonulak
added a commit
that referenced
this pull request
Jul 13, 2025
commit ab9ba27 Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Sat Jul 12 01:39:36 2025 -0400 Feat/redesign programs page (#793) * Feat: Redesign programs page with new Hero, ProgramCard, and CTA components * feat: update Programs page layout with new HeroArea and ProgramCard integration --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> commit 39d578b Author: Jerome Hardaway <jerome@vetswhocode.io> Date: Sat Jul 12 00:32:49 2025 -0400 Program pages (#792) * Add core programs for veterans: Core Curriculum, Mentorship, Mission-Ready, and Studio - Created new MDX files for Core Curriculum, Mentorship Program, Mission-Ready Platform, and Studio. - Implemented program overview, outcomes, and how it works sections in each program's MDX file. - Developed a programs index page to display all programs with descriptions and links. - Added dynamic routing for individual program pages using Next.js. * chore: add @types/mdx dependency and update yarn.lock - Added @types/mdx version 2.0.13 to package.json. - Cleaned up yarn.lock by removing unused @emotion dependencies and ensuring proper versioning for existing packages. * Update src/pages/programs/[slug].tsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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 pull request introduces new components and refactors the
Programspage to enhance its design and usability. Key changes include the addition of reusable components (CTA,Hero, andProgramCard) and the restructuring of theProgramspage to use these components for improved modularity and visual appeal.New Components
src/components/cta/index.tsx: Added aCTAcomponent with a call-to-action section encouraging users to apply to the program.src/components/hero/index.tsx: Introduced aHerocomponent for displaying a visually appealing header section with program details.src/components/program-card/index.tsx: Created aProgramCardcomponent to display individual program information in a card format, including title, description, and a link to learn more.Refactoring of
ProgramsPagesrc/pages/programs.tsx: Replaced inline program card implementation with the newProgramCardcomponent and added theHeroAreasection to improve page structure. Updated text content and layout for better readability and user experience. [1] [2]