website icon indicating copy to clipboard operation
website copied to clipboard

Impact Site Update

Open smalex-z opened this issue 9 months ago • 2 comments

Copy from PR #783 , Moved changes to a internal ACM branch due to change in personnel

@Brendan-Staines Please take over from here- @doanneda Please advise if you have any

Overview

Related to #780

Deploy Preview:

Changes

  • All changes from PR #783 (Credit to RohanAdwankar)
  • Merge conflict fixed
  • PR #784 merged into this to fix the officer parsing errors
  • EsLint Errors to be fixed

Testing

  • Check Netlify preview

Possible Changes

  • ESLint errors to be fixed

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 avatar Apr 29 '25 07:04 smalex-z

run npm run lint to see the lint errors. Fix the rest of the errors so that it will pass the checks.

smalex-z avatar Apr 29 '25 07:04 smalex-z

Two things before we merge: image "nice ty!! could this part be adjusted? it looks a bit odd being at the bottom left lol, maybe moving it to the middle right?" - Impact

Also, remove Rohan and replace w/ Anne Do as Impact Advisor

smalex-z avatar May 02 '25 19:05 smalex-z