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

26 about us registration #29

Closed
wants to merge 5 commits into from
Closed

26 about us registration #29

wants to merge 5 commits into from

Conversation

brendanshen24
Copy link

Describe your changes

Implemented website from the 'About us' section down to the 'Registration' section

  1. Imported the Vollkorn font in globals.css.
  2. Modified tailwind.config.ts to expand on the theme section.
  3. Created AboutUs, DesignerFriendly, and Register component files.
  4. Made the layout change based on the mobile and desktop breakpoints.

Issue ticket number and link

#26

Testing Steps

  1. checkout to 26-about-us-registration
  2. run npm install to update dependencies
  3. run the development server.

Brendan Shen added 2 commits April 5, 2024 21:30
Initial commit. Implemented AboutUs, DesignerFriendly, and Register s…
9dfff4c 
…ections. Modified tailwind.config.ts to include some more themes and imported the Vollkorn font in globals.css. Layout changes based on the mobile and desktop breakpoints.
Fixed some minor stuff related to linting.
a73e81a
@netlify Netlify
Copy link

netlify bot commented Apr 6, 2024 •
edited
Loading

✅ Deploy Preview for stormhacksdev ready!

Name Link
🔨 Latest commit db36148
🔍 Latest deploy log https://app.netlify.com/sites/stormhacksdev/deploys/66130375f973460008b9930f
😎 Deploy Preview https://deploy-preview-29--stormhacksdev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@brendanshen24 brendanshen24 linked an issue Apr 6, 2024 that may be closed by this pull request
Add "About us" all the way to "Registration" sections #26
Open
ChrisFong604 and others added 3 commits April 6, 2024 14:55
@ChrisFong604
moved sections from pages to separate folder called 'sections'
90323b9
@brendanshen24
Merge branch 'development' of https://github.com/sfusurge/stormhacks_…
b5aeefe 
…website into 26-about-us-registration

# Conflicts:
#	src/pages/index.js
@brendanshen24
Adjusted existing page elements and modified some tailwind styles to …
db36148 
…accommodate new styling. I also added the 'sections' directory into the content section of tailwind.config.ts.
@davidrobertlowe davidrobertlowe self-assigned this Apr 12, 2024
@kevins10 kevins10 self-requested a review April 12, 2024 06:58
@davidrobertlowe davidrobertlowe self-requested a review April 12, 2024 07:00
davidrobertlowe
davidrobertlowe approved these changes Apr 12, 2024
View reviewed changes
Copy link
Contributor

@davidrobertlowe davidrobertlowe 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 merging

@davidrobertlowe davidrobertlowe removed the request for review from kevins10 April 12, 2024 07:06
@kevins10 kevins10 self-requested a review April 12, 2024 07:24
@davidrobertlowe
Copy link
Contributor

was having issues merging this branch, so just merged the changes on another

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidrobertlowe davidrobertlowe davidrobertlowe approved these changes

@kevins10 kevins10 kevins10 approved these changes

Assignees

@davidrobertlowe davidrobertlowe

@kevins10 kevins10

@jennaolee jennaolee

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add "About us" all the way to "Registration" sections
5 participants
@brendanshen24 @davidrobertlowe @kevins10 @jennaolee @ChrisFong604