Skip to content

Comments

seattlecolleges.github.io_1_122_add-student-bade-habib#139

Open
JungDefiant wants to merge 3 commits intoSeattleColleges:mainfrom
JungDefiant:add-student-bade-habib
Open

seattlecolleges.github.io_1_122_add-student-bade-habib#139
JungDefiant wants to merge 3 commits intoSeattleColleges:mainfrom
JungDefiant:add-student-bade-habib

Conversation

@JungDefiant
Copy link

Summary & Changes 📃

  • Resolves: 122

  • Summary: Adds Bade Habib to the student page.

    • 🔨 What does this issue fix? Adds Bade Habib to the student page.
    • 👀 What is the expected behavior? Bade Habib appears on the profile page for seattlecolleges.github.io.
    • 🗨️ Any relevant technical details? None.
  • 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.

Copy link
Contributor

@jmathew12 jmathew12 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 good!

Image

@endlesslupita
Copy link

PR Review: Nice documentation, including the screenshot and all the components of the PR template. You can resolve conflicts in the GitHub web editor. I found that was easier than using VS Code.

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.

4 participants