Skip to content

Comments

Add Dara Sen to students list closes #122#131

Open
DS-0277 wants to merge 1 commit intoSeattleColleges:mainfrom
DS-0277:add-student-dara-sen
Open

Add Dara Sen to students list closes #122#131
DS-0277 wants to merge 1 commit intoSeattleColleges:mainfrom
DS-0277:add-student-dara-sen

Conversation

@DS-0277
Copy link

@DS-0277 DS-0277 commented Feb 14, 2026

Hi, I have added my profile to the students page. Closes #122.

Summary & Changes 📃

  • Resolves: #issue-number

  • Summary: (Briefly describe what this PR does)

    • 🔨 What does this issue fix?
    • 👀 What is the expected behavior?
    • 🗨️ Any relevant technical details?
  • Changes:

    • ✅ List key changes made
    • 🛠️ Mention breaking changes (if any)
    • 🔗 Link relevant discussions/issues
    • 📝 Additional info to assist developers & reviewers

Screenshots / Visual Aids 🔎

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

Expand ⬇️

How to Test 🧪

  1. Steps to Reproduce:
    • Step 1: ...
    • Step 2: ...
  2. Expected Behavior: (Describe what should happen)
  3. Actual Behavior (if bug): (Describe what happens instead)

Checklist ✅

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

@LiteWilliamNguyen
Copy link
Contributor

LiteWilliamNguyen commented Feb 14, 2026

It looks like the error is occurring because the photo hasn’t been added to the project folder:
SeattleColleges.github.io/src/images/students
Please add the image to that directory using the exact filename: Dara-Sen.webp.
The rest of code in JSON is working properly other than that issue.

image

@DS-0277
Copy link
Author

DS-0277 commented Feb 14, 2026

Thanks for letting me know. I have just added the file 'Dara-Sen.webp' in the directory.

Copy link

@Eric-Cashman Eric-Cashman left a comment

Choose a reason for hiding this comment

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

Looks good! Nicely done!

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.

Students Page: Add yourself to the Students Page

3 participants