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

Temporary changes #55

Merged
merged 2 commits into from
Nov 30, 2023
Merged

Temporary changes #55

merged 2 commits into from
Nov 30, 2023

Conversation

samderanova
Copy link
Contributor

@samderanova samderanova commented Nov 30, 2023 •
edited
Loading

  • Hide the mentor and volunteer section as apps aren't ready yet
  • Add bottom margin to the koi fish at the bottom of the about section
  • Update about section information

@samderanova
Temporary changes:
20713a6 
- Hide the mentor and volunteer section as apps aren't ready yet
- Add bottom margin to the koi fish at the bottom of the about section
@samderanova samderanova requested a review from a team November 30, 2023 21:29
@samderanova samderanova linked an issue Nov 30, 2023 that may be closed by this pull request
Temporarily hide mentor and volunteer sections #54
Closed
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 30, 2023 •
edited
Loading

Deploy preview for irvinehacks-site-2024-sanity ready!

Name Sanity Studio
Preview Visit Preview
Commit 1a17e36

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 30, 2023 •
edited
Loading

Deploy preview for irvinehacks-site-2024 ready!

Name IrvineHacks 2024 Site
Preview Visit Preview
Commit 1a17e36

Bl20052005
Bl20052005 approved these changes Nov 30, 2023
View reviewed changes
Copy link
Contributor

@Bl20052005 Bl20052005 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

waalbert
waalbert approved these changes Nov 30, 2023
View reviewed changes
Copy link
Contributor

@waalbert waalbert left a comment

Choose a reason for hiding this comment

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

LGTM

@samderanova samderanova merged commit daec239 into main Nov 30, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bl20052005 Bl20052005 Bl20052005 approved these changes

@waalbert waalbert waalbert approved these changes

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

Successfully merging this pull request may close these issues.

Temporarily hide mentor and volunteer sections
3 participants
@samderanova @waalbert @Bl20052005