Skip to content

Conversation

@Ethjin8
Copy link
Contributor

@Ethjin8 Ethjin8 commented Oct 31, 2025

Overview

Resolves #864

Deploy Preview: https://deploy-preview-869--jovial-pasteur-581b4a.netlify.app/

Changes

  • Rerouted invite link of the main ACM Discord server to uclaacm.com/discord
  • Changed icon link to /discord in SocialMedia.js file
  • Added redirect() function in next.config.js file, pointing to the server invite
  • No new dependencies required

Testing

  • Hover over the Discord icons on the website page to see if new link is there
image

Possible Changes

  • The Discord invite links for other ACM committees haven't been rerouted and are still there, which warrants future fixing

Checklist

  • Code follows the project's style guidelines.
  • Documentation has been updated where necessary.
  • All checks pass and deploy builds with no errors.

@smalex-z
Copy link
Contributor

https://deploy-preview-869--jovial-pasteur-581b4a.netlify.app/

^^ Netlify Deploy Preview

@Ethjin8
Copy link
Contributor Author

Ethjin8 commented Oct 31, 2025

Thanks I couldn't figure out the right link for some reason

@smalex-z smalex-z self-requested a review October 31, 2025 23:02
Copy link
Contributor

@smalex-z smalex-z left a comment

Choose a reason for hiding this comment

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

lgtm! congrats on your first PR

@Ethjin8 Ethjin8 merged commit 6c130e3 into main Nov 1, 2025
8 checks passed
@Ethjin8 Ethjin8 deleted the reroute-discord-link branch November 1, 2025 00:11
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.

Reroute Discord Link

3 participants