Skip to content

Conversation

@derekxu04
Copy link
Contributor

@derekxu04 derekxu04 commented Jul 4, 2024

Changes:

  • Fixed assets not showing up on some pages
  • Refactored data.ts format
  • Created components for Layout, PageBody and PageTitle
  • Some fixes to hamburger and pages
  • Changed about description to be more comprehensive
  • Changed breadcrumb -> this is probably the biggest change (I thought the previous breadcrumb didn't really make sense, only worry is this might be too repetitive given the logo above? not sure)

image

@derekxu04 derekxu04 requested a review from andrew-l3 July 22, 2024 08:52
@derekxu04 derekxu04 changed the title DRAFT: CW2-31 refactor code CW2-31 refactor code Jul 22, 2024
@derekxu04 derekxu04 changed the title CW2-31 refactor code [CW2-31] refactor code Jul 22, 2024
Copy link
Contributor

@andrew-l3 andrew-l3 left a comment

Choose a reason for hiding this comment

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

Seems good. Going down the refactoring path, we could probably simplify components in files like execs-directors-subcoms.tsx and constitution.tsx (i.e. add a component and move its data to an existing .ts file like before) but that can be done down the line.

@derekxu04 derekxu04 merged commit a4b12d8 into master Jul 25, 2024
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