Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Comments

nextwavedev_1_8_add-shift-geekgirlcon-lictonsprings-to-portfolio#9

Merged
JesseCaddell merged 2 commits intomainfrom
portfolio-update/licton-geekgirl-shift
May 12, 2025
Merged

nextwavedev_1_8_add-shift-geekgirlcon-lictonsprings-to-portfolio#9
JesseCaddell merged 2 commits intomainfrom
portfolio-update/licton-geekgirl-shift

Conversation

@beimnettes
Copy link
Contributor

Summary & Changes 📃

  • Resolves: #8

  • Summary: This PR adds three new student-built projects to the Portfolio Page:

    • 🔨 Adds Shift App, GeekGirlCon Registration, and Licton Springs Review sections
    • 👀 Each project displays title, image, description, and external links
    • 🗨️ Supports ongoing improvements to showcase real-world student work
  • Changes:

    • ✅ Updated PortfolioPage.js with new project entries
    • ✅ Added 3 screenshots to src/images/projects/
    • ✅ Imported and styled images consistently with existing layout
    • 📝 Linked to GitHub and live demo URLs

Screenshots / Visual Aids 🔎

📌 Required for: UI changes, layout updates, or bug fixes.

Expand ⬇️ shift-app Screenshot 2025-05-04 at 9 14 54 PM

How to Test 🧪

  1. Steps to Reproduce:
    • Run npm start
    • Navigate to /portfolio
    • Scroll to the bottom
  2. Expected Behavior:
    • All 3 new project cards render with correct data and images
    • All external links (GitHub + live demo) work as expected
  3. Actual Behavior (if bug):
    • N/A – All tested locally

Checklist ✅

  • I have tested this PR locally and it works as expected.
  • This PR resolves an issue (Resolves #8).
  • Reviewers, assignees (self), tags, and labels are correctly assigned.
  • Squash commits and enable auto-merge if approved.

@beimnettes beimnettes self-assigned this May 5, 2025
Copy link

@MiguelQuintana1 MiguelQuintana1 left a comment

Choose a reason for hiding this comment

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

Good Job on these changes, the website looks fantastic, visually very pleasant. Good Job!!

GMT20250505-055605_Clip_Miguel.Quintana.s.Clip.05_04_2025.mp4

Copy link

@gitbiruk2010 gitbiruk2010 left a comment

Choose a reason for hiding this comment

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

Nicely done @beimnettes! embedded links are hot and added projects render correctly.
Test Demo:

nextw-pr-9-review.mp4

Copy link
Contributor

@AzariasANeals AzariasANeals left a comment

Choose a reason for hiding this comment

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

Everything looks fantastic! All the images load properly and are in style. All of the links work as well. Looks ready to be merged.
expo-app-portfolio-screenshot
geekgirlcon-portfolio-screenshot
licton-spring-review-screenshot

Copy link

@cshimm cshimm left a comment

Choose a reason for hiding this comment

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

The layout and styling looks amazing! I did notice, however, some inaccuracies with the GeekGirlCon description so I left some comments about that.

Copy link

@cshimm cshimm left a comment

Choose a reason for hiding this comment

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

Awesome job!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue: NextWaveDev_91_Add Shift App, GeekGirlCon, and Licton Springs Review to Portfolio Page

6 participants