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

Mc button #46

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

Conversation

MCaldwell-42
Copy link

@MCaldwell-42 MCaldwell-42 commented Mar 17, 2020

nashvillefcc#23

Here is a link to the issue. other than the style modes it is also set up to be dynamic based on what the button says allowing some deviation, while still taking users to the correct link.

@Hellrungj it doesn't appear I can assign reviewers so here is a tag.

This PR is set to merge to master and I'm a bit confused by code repo and issues repo but...trusting y'all have it sorted.

Hellrungj and others added 25 commits March 1, 2020 21:59
…ndex page

Also add a plug-in for handling svgs in gastby and react-feather
Add a svgs folder to hold all the svgs images plus the gatsby needs to a stand alone directory
react-feather was already using used in the old theme but on it's a main package
Additions to the index page include Speakers, Venue and Sponsors sections along with a call to action for Sponsors or Volunterrs
Add the following pages:
- contact
- schedule
- speakers
Reviewed and approved.
Mirgated most of the site to scss rather than react styles
Reviewed and approved
Readme update
Add a content about Branches, Pull Request, How to contact, what should I know
Add the folowing assists:
- ugjs-skyline.svg
- map-pin.png
- ninja.png
- superhero.png
- twitter-icon.png
- ugjs-logo-large.png
- ugjs-logo-sm.png
- ugjs-logo-sm-color.png
Added Helmet Plugin and Fonts using SASS imports - Checked and approved -- this is a breaking change.
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.

4 participants