Rcint 7530 audio setting messy code #781
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
embbnux
reviewed
May 23, 2018
| if (newProps.inputDeviceId !== this.props.inputDeviceId) { | ||
| if (newProps.inputDeviceId !== this.props.inputDeviceId || all( | ||
| device => device.deviceId !== this.state.inputDeviceId, | ||
| newProps.availableInputDevices |
There was a problem hiding this comment.
Can we validate and update inputDeviceId when availableInputDevices are changed in audio setting module?
There was a problem hiding this comment.
The inputDeviceId from AudioSetting module is correct. The code is to correct the component inputDeviceId state itself.
embbnux
approved these changes
May 25, 2018
ruleechen
added a commit
to ruleechen/ringcentral-js-widgets
that referenced
this pull request
May 27, 2018
* Modify mockApi to specify http method name * disable log btn in call list when the item is fax messsage (ringcentral#775) * disable log btn in call list when the item is fax messsage * should show button on call log notification when have related function * Fix mock API for meeting scheduler (ringcentral#777) * Mock Meeting & Service Info api (ringcentral#779) * mock API * no message * change switch disable color (ringcentral#778) * change switch disable color * remove setting page last two item line border (ringcentral#782) * remove setting page last two item line border * [feature] Wrap libphonenumber-js in @ringcentral-integration/phone-number package (ringcentral#783) * fix linkline icon postion to alignment right (ringcentral#784) * fix linkline icon postion to alignment right * modify logout icon size * Modify style of log icon in calls tab (ringcentral#785) * Modify style of log icon * fix dialerUI hooks sync (ringcentral#786) * Rcint 7530 audio setting messy code (ringcentral#781) * fix messy code showed when device is plugged out * code refine * update * update * fix failed tests in LogBasicInfo which is caused by letter case mismatch (ringcentral#789) * Update the style and text of DatePicker. (ringcentral#792) * fix search input exceeding width issue (ringcentral#791) * bugfix: user guide module cannot fall back to en-US automatically (ringcentral#787) * minor: make close button optional in AdpterCore (ringcentral#788) * fix SpinnerOverlay background-color translucence bug for safari (ringcentral#794) * fix tests in LogBasicInfo which is caused by letter case mismatch (ringcentral#793)
ruleechen
pushed a commit
that referenced
this pull request
Jul 27, 2018
* sync from upstream (#16) * Modify mockApi to specify http method name * disable log btn in call list when the item is fax messsage (#775) * disable log btn in call list when the item is fax messsage * should show button on call log notification when have related function * Fix mock API for meeting scheduler (#777) * Mock Meeting & Service Info api (#779) * mock API * no message * change switch disable color (#778) * change switch disable color * remove setting page last two item line border (#782) * remove setting page last two item line border * [feature] Wrap libphonenumber-js in @ringcentral-integration/phone-number package (#783) * fix linkline icon postion to alignment right (#784) * fix linkline icon postion to alignment right * modify logout icon size * Modify style of log icon in calls tab (#785) * Modify style of log icon * fix dialerUI hooks sync (#786) * Rcint 7530 audio setting messy code (#781) * fix messy code showed when device is plugged out * code refine * update * update * fix failed tests in LogBasicInfo which is caused by letter case mismatch (#789) * Update the style and text of DatePicker. (#792) * fix search input exceeding width issue (#791) * bugfix: user guide module cannot fall back to en-US automatically (#787) * minor: make close button optional in AdpterCore (#788) * fix SpinnerOverlay background-color translucence bug for safari (#794) * fix tests in LogBasicInfo which is caused by letter case mismatch (#793) * code specification * refactor the lastcallend func * eslinted * fix(blank avatar bug fixes rcint-8106) * only the avatar is null fetch the avatar again * to update to the lasted code with merge control * change the merge info com to a lifecircle com * update * update
huanhulan
pushed a commit
that referenced
this pull request
Aug 1, 2018
* sync from upstream (#16) * Modify mockApi to specify http method name * disable log btn in call list when the item is fax messsage (#775) * disable log btn in call list when the item is fax messsage * should show button on call log notification when have related function * Fix mock API for meeting scheduler (#777) * Mock Meeting & Service Info api (#779) * mock API * no message * change switch disable color (#778) * change switch disable color * remove setting page last two item line border (#782) * remove setting page last two item line border * [feature] Wrap libphonenumber-js in @ringcentral-integration/phone-number package (#783) * fix linkline icon postion to alignment right (#784) * fix linkline icon postion to alignment right * modify logout icon size * Modify style of log icon in calls tab (#785) * Modify style of log icon * fix dialerUI hooks sync (#786) * Rcint 7530 audio setting messy code (#781) * fix messy code showed when device is plugged out * code refine * update * update * fix failed tests in LogBasicInfo which is caused by letter case mismatch (#789) * Update the style and text of DatePicker. (#792) * fix search input exceeding width issue (#791) * bugfix: user guide module cannot fall back to en-US automatically (#787) * minor: make close button optional in AdpterCore (#788) * fix SpinnerOverlay background-color translucence bug for safari (#794) * fix tests in LogBasicInfo which is caused by letter case mismatch (#793) * code specification * refactor the lastcallend func * eslinted * bug fixes * 🐛 (rcint-8156) bug fixes(merge two call then the call avatar will lose) * code update to upstream
jarvislin94
pushed a commit
to jarvislin94/ringcentral-js-widgets
that referenced
this pull request
Aug 2, 2018
* Modify mockApi to specify http method name * disable log btn in call list when the item is fax messsage (ringcentral#775) * disable log btn in call list when the item is fax messsage * should show button on call log notification when have related function * Fix mock API for meeting scheduler (ringcentral#777) * Mock Meeting & Service Info api (ringcentral#779) * mock API * no message * change switch disable color (ringcentral#778) * change switch disable color * remove setting page last two item line border (ringcentral#782) * remove setting page last two item line border * [feature] Wrap libphonenumber-js in @ringcentral-integration/phone-number package (ringcentral#783) * fix linkline icon postion to alignment right (ringcentral#784) * fix linkline icon postion to alignment right * modify logout icon size * Modify style of log icon in calls tab (ringcentral#785) * Modify style of log icon * fix dialerUI hooks sync (ringcentral#786) * Rcint 7530 audio setting messy code (ringcentral#781) * fix messy code showed when device is plugged out * code refine * update * update * fix failed tests in LogBasicInfo which is caused by letter case mismatch (ringcentral#789) * Update the style and text of DatePicker. (ringcentral#792) * fix search input exceeding width issue (ringcentral#791) * bugfix: user guide module cannot fall back to en-US automatically (ringcentral#787) * minor: make close button optional in AdpterCore (ringcentral#788) * fix SpinnerOverlay background-color translucence bug for safari (ringcentral#794) * fix tests in LogBasicInfo which is caused by letter case mismatch (ringcentral#793)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.