Skip to content

Conversation

@smalex-z
Copy link
Contributor

@smalex-z smalex-z commented Sep 9, 2025

Overview

Resolves #NA

Deploy Preview

  • Note banner rendering will fail until this PR is merged in, since it's fetching directly from the website

Changes

  • GM page updated to CSR to avoid hydration issues with countdown.
  • Data parsed from the ACM Website Data Hub spreadsheet now (updated w/ new values)
  • Banner link added in the Current GM field
  • Static images moved to separate folder in maintain organization

Testing

Possible Changes

Checklist

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

Copy link

@bzhang102 bzhang102 left a comment

Choose a reason for hiding this comment

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

LGTM lil bro as long as ur sure the 404 for the banner resolves after merge :)

@smalex-z smalex-z merged commit 9bf86f0 into main Sep 10, 2025
8 checks passed
@smalex-z smalex-z deleted the gm-25-updates branch September 10, 2025 07:59
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.

3 participants