Skip to content

Optimize and cleanup app for rollout PART 2#52

Merged
Im-Madhur-Gupta merged 12 commits intomainfrom
camille/eng-524-optimize-and-cleanup-app-for-rollout
Jan 12, 2026
Merged

Optimize and cleanup app for rollout PART 2#52
Im-Madhur-Gupta merged 12 commits intomainfrom
camille/eng-524-optimize-and-cleanup-app-for-rollout

Conversation

@Camillebzd
Copy link
Contributor

@Camillebzd Camillebzd commented Jan 12, 2026

This one contains:

  • style cleanup
  • default export converted to name export

@linear
Copy link

linear bot commented Jan 12, 2026

ENG-524 Optimize and cleanup app for rollout

  • In a lot of our hooks used to retrieve data for the components, we don't have any limits. Can be a problem if the tab stays open for a long duration as the amount of data will grow indefinitely.
  • in useBlockTracker (called without nothing so default infinite returns)
  • in useTransferEvents (the var is set to Infinity)
  • No limit in data stored in our context events-context.
  • Any values in px, hex color strings can be cleaned up such that we use rem values, CSS variables (if its a common hex color)
  • For some components we have default exports instead of named exports, I'd suggest we modify it to named exports for all
  • Optional: find a way to keep the same behaviour of the component with rolling data (delete the old one while accepting the new one)

@vercel
Copy link

vercel bot commented Jan 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
execution-events-example Ready Ready Preview, Comment Jan 12, 2026 3:23pm

@Im-Madhur-Gupta Im-Madhur-Gupta merged commit 43e8ac9 into main Jan 12, 2026
6 checks passed
@Im-Madhur-Gupta Im-Madhur-Gupta deleted the camille/eng-524-optimize-and-cleanup-app-for-rollout branch January 12, 2026 17: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.

2 participants