Skip to content

Comments

seattlecolleges.github.io_122_add-hyejin-lee-to-students#127

Merged
bcko merged 1 commit intoSeattleColleges:mainfrom
HJLee82:add-student-hyejin-lee
Feb 20, 2026
Merged

seattlecolleges.github.io_122_add-hyejin-lee-to-students#127
bcko merged 1 commit intoSeattleColleges:mainfrom
HJLee82:add-student-hyejin-lee

Conversation

@HJLee82
Copy link
Contributor

@HJLee82 HJLee82 commented Feb 11, 2026

Summary & Changes 📃

  • Resolves: #122

  • Summary: (Briefly describe what this PR does)

    • 🔨 Adds Hyejin Lee to the Students Page.
    • 👀 Only src/students.json was modified. No existing entries were changed or removed.
  • Changes:

    • ✅ Added a new student object at the end of the students array in src/students.json.

Screenshots / Visual Aids 🔎

Screenshot01 Medium

How to Test 🧪

  1. Steps to Reproduce:
    • Step 1: Pull the branch locally
    • Step 2: Run the project
    • Step 3: Navigate to the Students page
  2. Expected Behavior: Hyejin Lee appears at the bottom of the Students list.

Checklist ✅

@bcko bcko self-requested a review February 11, 2026 23:22
Copy link
Contributor

@bcko bcko left a comment

Choose a reason for hiding this comment

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

@HJLee82 Great job! Can you update your Pull Request to include a screenshot or a video recording of your change? thanks

Copy link

@nhapham03 nhapham03 left a comment

Choose a reason for hiding this comment

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

LGTM! Your work shows up on my end, works as expected 👍

Image

@bcko bcko enabled auto-merge (squash) February 20, 2026 02:48
Copy link
Contributor

@bcko bcko left a comment

Choose a reason for hiding this comment

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

Great job

@bcko bcko merged commit fc891a3 into SeattleColleges:main Feb 20, 2026
0 of 3 checks passed
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