Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Completed Ramp Up Task #1 #34

Closed
wants to merge 1 commit into from
Closed

feat: Completed Ramp Up Task #1 #34

wants to merge 1 commit into from

Conversation

sab115
Copy link
Contributor

@sab115 sab115 commented Feb 3, 2025 •
edited
Loading

Developer: Sabrina Huang

Closes #{ISSUE NUMBER HERE}
#1

Pull Request Summary

Completed Ramp Up Task 1

Modifications

  • create sample page
  • added a button that directs back to the home page

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

{put screenshots of your change, or even better a screencast displaying the functionality}
Screenshot 2025-02-02 at 4 32 36 PM

@sab115 sab115 requested review from aarav27 and ktaschek February 3, 2025 00:34
aarav27
aarav27 approved these changes Feb 3, 2025
View reviewed changes
Copy link
Collaborator

@aarav27 aarav27 left a comment

Choose a reason for hiding this comment

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

Nice job Sabrina, LTGM!

@aarav27 aarav27 closed this Feb 3, 2025
@ktaschek ktaschek deleted the 1-sab115 branch February 5, 2025 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aarav27 aarav27 aarav27 approved these changes

@ktaschek ktaschek Awaiting requested review from ktaschek

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sab115 @aarav27