-
Notifications
You must be signed in to change notification settings - Fork 40
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(meeting): fetching captcha details within meeting object #337
Conversation
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.
LGTM! Tested locally working fine. Captcha details were populated
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.
Can we have details of the JIRA that this is fixing as well as the reason why the change is required?
Please see the comments.
src/MeetingsSDKAdapter.js
Outdated
}), | ||
})); | ||
} else { | ||
logger.info('MEETING', ID, 'joinMeeting()', 'Password succesfully verified'); |
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.
There is a typo in the log. Please fix it.
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.
Addressed
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.
Please add the JIRA link to the description for tracking.
## [1.112.5](v1.112.4...v1.112.5) (2024-06-28) ### Bug Fixes * **meeting:** fetching captcha details within meeting object ([#337](#337)) ([34a8a18](34a8a18))
🎉 This PR is included in version 1.112.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Storing the response for verifyPassword() and fetch details like invalidPassword, failureReason and requiredCaptcha and update the meeting object with these details.
Updated verifyPassword() function to accept captcha as a parameter.
Added a new function to refresh the captcha when needed
JIRA: https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-521834
Tests Performed:
Meeting Widget Testing
https://github.com/webex/sdk-component-adapter/assets/65543166/8b6b8ee4-7676-48b2-a122-32a487a7b1d1
Logs
Widgets_Testing_logs.log