New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix case when passcode was lost #4535
Conversation
@@ -245,10 +247,11 @@ export const HomeScreen = (props: HomeScreenProps): JSX.Element => { | |||
label={'Passcode'} | |||
placeholder={'Enter a meeting passcode'} | |||
onChange={(_, newValue) => { | |||
const meetingId = callLocator && 'meetingId' in callLocator ? callLocator.meetingId : ''; | |||
const localMeetingId = meetingId ? meetingId : ''; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this line needed? meetingId
can just be used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is just undefined check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right - but why have it? You have the check again below when localMeetingId ?
is called
@azure/communication-react jest test coverage for stable.
|
@azure/communication-react jest test coverage for beta.
|
…://github.com/Azure/communication-ui-library into ruslanz/meetingid-passcode-lost-url-parameter
Calling bundle size is decreased✅.
|
Chat bundle size is decreased✅.
|
CallWithChat bundle size is decreased✅.
|
* fix case when passcode was lost in url based on input order
What
If I enter the passcode first, then enter the meetingId, the passcode isn't passed to the URL:
Why
How Tested
Process & policy checklist
Is this a breaking change?