Skip to content

Conversation

@apsantiso
Copy link
Contributor

No description provided.

@apsantiso apsantiso self-assigned this Sep 17, 2025
@apsantiso apsantiso requested a review from CandelR September 17, 2025 00:24
const { uuid, email } = user || {};
const isUserAnonymous =
(!joinCallDto?.anonymousId || joinCallDto?.anonymous) ?? !user;
(!!joinCallDto?.anonymousId || joinCallDto?.anonymous) ?? !user;
Copy link
Contributor

Choose a reason for hiding this comment

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

@apsantiso Check that this condition matches what we want. I changed it because I think it was wrong
!joinCallDto?.anonymousId -> !!joinCallDto?.anonymousId

@apsantiso apsantiso force-pushed the feat/allow-anonymousId-from-frontend branch from 3fd346f to ffe2aaa Compare September 17, 2025 14:03
@apsantiso apsantiso merged commit b90c914 into master Sep 17, 2025
4 of 5 checks passed
@apsantiso apsantiso deleted the feat/allow-anonymousId-from-frontend branch September 17, 2025 15:49
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