Skip to content

Conversation

@Udit-takkar
Copy link
Contributor

@Udit-takkar Udit-takkar commented Jan 18, 2026

What does this PR do?

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • N/A I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?


Summary by cubic

Reduced unnecessary re-renders in the Booker by replacing object-based effect dependencies with primitives and tidying imports. This makes state transitions more stable with no user-facing changes.

  • Refactors
    • Updated effect dependency from the event object to event.isPending in Booker.tsx.
    • Consolidated and reordered imports.

Written for commit 7f27091. Summary will update on new commits.

@graphite-app graphite-app bot added core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO labels Jan 18, 2026
@graphite-app graphite-app bot requested a review from a team January 18, 2026 10:08
return setBookerState("booking");
}, [event, selectedDate, selectedTimeslot, setBookerState, skipConfirmStep, layout, isInstantMeeting]);
}, [
event.isPending,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the only code change in this PR. We should use event.isPending instead of event

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants