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

Moved about pages into the front end #640

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from

Conversation

rjmara
Copy link
Contributor

@rjmara rjmara commented Nov 9, 2022

Description

Previously we had stored many of the footer pages in the backend. This change allows us to free up clutter in the backend. Additionally, it allows anybody to make changes much more efficiently than when it was on the backend. The only way to see the changes to your code was to open the html file and constantly reload the page to see the changes you made. Instead of that tedious process, you can just use an npm command to see your changes in real time.

The pages that were added were:

  • about
  • collections
  • team (updated to current team)
  • privacy policy
  • terms of service
  • support

Additional changes were done to routes.js in order to use the pages in the frontend rather than the backend; fontawesome packages were added to add icons for some of the about images including brand images like linkedin and github for team members that had websites linked in their respective cards; added another folder to hold images of team members; a couple of changes were done to App.css because it was difficult to get the same effect using bootstrap classes. However, most of the styling changes were done using bootstrap because the css files are global across all MYR pages.

Preview

About page header:
image

Related Issue

Issue #216

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.

1 participant