Skip to content

Comments

Add student: Reid SantaBarbara#137

Open
olympusmons1256 wants to merge 3 commits intoSeattleColleges:mainfrom
olympusmons1256:add-student-reid-santabarbara
Open

Add student: Reid SantaBarbara#137
olympusmons1256 wants to merge 3 commits intoSeattleColleges:mainfrom
olympusmons1256:add-student-reid-santabarbara

Conversation

@olympusmons1256
Copy link

Closes: #122

Summary & Changes

Added my student profile to the end of students.json
Uploaded my photo as Reid-SantaBarbara.webp

How to Test

Check out this branch:
git checkout add-student-reid-santabarbara

Run

  • npm install && npm start
  • Navigate to the Students page.
  • Confirm my profile appears at the end of the list and my photo loads correctly. And no console or JSON errors.

Checklist

[x] Tested locally and works as expected.
[x] PR resolves issue #122.
[x] Reviewers, assignees(self), tags, and labels are correctly assigned.
[x] Squash commits and enable auto-merge if approved.

Copy link

@elbi3 elbi3 left a comment

Choose a reason for hiding this comment

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

I tested this locally and it looks like all your information is here, and the Sims photo is painting to the page. Great work.

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