Skip to content

Conversation

@No767
Copy link
Member

@No767 No767 commented Dec 21, 2024

Summary

This PR is focused on cleaning up any unused files, imports, and others. This is done in preparation of the new design, which will need completely new code and looks.

Also migrated the logo to the public folder so it doesn't get included in the build process. And also reverted back to React 18 due to peer dep conflicts.

Types of changes

What types of changes does your code introduce to the UC Merced's ACM Chapter Website?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (Updates to README.md, the documentation, etc)
  • Other (if none of the other choices apply)

Checklist

Put an x in the boxes that apply

  • If code changes were made then they have been tested.
  • All workflows pass with my new changes
  • [ x This PR does not address a duplicate issue or PR

@netlify
Copy link

netlify bot commented Dec 21, 2024

Deploy Preview for ucmacm ready!

Name Link
🔨 Latest commit b891188
🔍 Latest deploy log https://app.netlify.com/sites/ucmacm/deploys/67665a4b917d740008c46efa
😎 Deploy Preview https://deploy-preview-585--ucmacm.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.

@No767 No767 changed the title Remove any unnecessary code in preperation of new design Remove any unnecessary code in preparation of new design Dec 21, 2024
@sonarqubecloud
Copy link

@No767 No767 merged commit 4dd5309 into react-rewrite Dec 21, 2024
11 checks passed
@No767 No767 deleted the noelle/cleanup-for-rewrite branch December 21, 2024 06:05
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.

2 participants