Skip to content

Conversation

@korrica1-design
Copy link

@korrica1-design korrica1-design commented Jan 26, 2026

… NaN userId.

What does this PR do?

Visual Demo (For contributors especially)

A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).

Video Demo (if applicable):

  • Show screen recordings of the issue or feature.
  • Demonstrate how to reproduce the issue, the behavior before and after the change.

Image Demo (if applicable):

  • Add side-by-side screenshots of the original and updated change.
  • Highlight any significant change(s).

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • 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.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings
  • My PR is too large (>500 lines or >10 files) and should be split into smaller PRs

Summary by cubic

Prevented NaN userId when assigning hosts in team event types. Email invite options now map to userId 0 for consistent member assignment.

  • Bug Fixes
    • Set userId to 0 when the option is an email invite or value is not a number, matching the Linear issue requirement to correctly handle email-invite members.

Written for commit a7e074b. Summary will update on new commits.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Jan 26, 2026
@vercel
Copy link

vercel bot commented Jan 26, 2026

@korrica1-design is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@korrica1-design
Copy link
Author

I've resolved the merge conflicts and applied a fix for the NaN userId issue in AddMembersWithSwitch.tsx. This should resolve the crashes occurring during team event-type assignments via email. Ready for review! /attempt #23467

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

community Created by Linear-GitHub Sync size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant