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.

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

Updated content from codeRIT to BrickHack #55

Merged
merged 9 commits into from
Jul 1, 2021
Merged

Updated content from codeRIT to BrickHack #55

merged 9 commits into from
Jul 1, 2021

Conversation

skyegallup
Copy link

  • Updated club name, images, links, etc.
  • Replaced body content to better reflect our club's goals
  • Temporarily commented out Facebook group link and disabled "join our Discord" button

Fixes #54

@skyegallup skyegallup requested a review from peterkos June 29, 2021 01:43
@skyegallup skyegallup self-assigned this Jun 29, 2021
@peterkos
Copy link

I think we should add a little blurb above "more info coming soon" that highlights that BrickHack is our main event just to make the distinction clear for now!

Copy link

@peterkos peterkos left a comment

Choose a reason for hiding this comment

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

LGTM! see comment before this review

@skyegallup
Copy link
Author

@peterkos Does this work?
image

@peterkos
Copy link

peterkos commented Jul 1, 2021

@clairegallup I think we should add a link to BrickHack there as it's most likely people will click that to go to the site vs. scroll down more. I don't personally see the harm in putting the link again there (while also keeping it in the Events section below), thoughts?

@skyegallup
Copy link
Author

That makes sense. I'll do that and push my changes for a final review.

@skyegallup
Copy link
Author

Does this work?
image

@skyegallup skyegallup merged commit b997857 into master Jul 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants