Fixes done for record button bug #14
Merged
Conversation
This file contains 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
soumik-mukherjee
added a commit
that referenced
this pull request
Apr 16, 2020
* Documentation Fix: Create the bare minimums (#12) * Created the readme file * Update issue templates Added the default GH templates for bug reports and features * Added PR templates * Workaround for GH problem: wont pick multiple PR templates * Removing workaround to test alternate GH does not seem to pickup multiple PR templates. This file was added temporarily to validate this issue (indeed it doesn't and by default only picks up `.github/pull_request_template.md`). Looks like the only way to get multiple templates working is by using URL query parameters as discussed (here)[https://help.github.com/en/github/managing-your-work-on-github/about-automation-for-issues-and-pull-requests-with-query-parameters] * Updated PR templates to remove front matter headings * Publishing readme content * Fixed minor markdown defects * Minor content edits * Fixes done (#14)
soumik-mukherjee
added a commit
that referenced
this pull request
Apr 16, 2020
* Upstream merge for WIP bug branch (#15) * Documentation Fix: Create the bare minimums (#12) * Created the readme file * Update issue templates Added the default GH templates for bug reports and features * Added PR templates * Workaround for GH problem: wont pick multiple PR templates * Removing workaround to test alternate GH does not seem to pickup multiple PR templates. This file was added temporarily to validate this issue (indeed it doesn't and by default only picks up `.github/pull_request_template.md`). Looks like the only way to get multiple templates working is by using URL query parameters as discussed (here)[https://help.github.com/en/github/managing-your-work-on-github/about-automation-for-issues-and-pull-requests-with-query-parameters] * Updated PR templates to remove front matter headings * Publishing readme content * Fixed minor markdown defects * Minor content edits * Fixes done (#14) * Added new Link comp and associated changes
soumik-mukherjee
added a commit
that referenced
this pull request
Apr 16, 2020
* Documentation Fix: Create the bare minimums (#12) * Created the readme file * Update issue templates Added the default GH templates for bug reports and features * Added PR templates * Workaround for GH problem: wont pick multiple PR templates * Removing workaround to test alternate GH does not seem to pickup multiple PR templates. This file was added temporarily to validate this issue (indeed it doesn't and by default only picks up `.github/pull_request_template.md`). Looks like the only way to get multiple templates working is by using URL query parameters as discussed (here)[https://help.github.com/en/github/managing-your-work-on-github/about-automation-for-issues-and-pull-requests-with-query-parameters] * Updated PR templates to remove front matter headings * Publishing readme content * Fixed minor markdown defects * Minor content edits * Fixes done (#14) * BugFix: Issue#9 download button not working (#16) * Upstream merge for WIP bug branch (#15) * Documentation Fix: Create the bare minimums (#12) * Created the readme file * Update issue templates Added the default GH templates for bug reports and features * Added PR templates * Workaround for GH problem: wont pick multiple PR templates * Removing workaround to test alternate GH does not seem to pickup multiple PR templates. This file was added temporarily to validate this issue (indeed it doesn't and by default only picks up `.github/pull_request_template.md`). Looks like the only way to get multiple templates working is by using URL query parameters as discussed (here)[https://help.github.com/en/github/managing-your-work-on-github/about-automation-for-issues-and-pull-requests-with-query-parameters] * Updated PR templates to remove front matter headings * Publishing readme content * Fixed minor markdown defects * Minor content edits * Fixes done (#14) * Added new Link comp and associated changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Fixes #13
Describe the changes proposed
The root cause is simply with a wrong expression used for callback/event bubbling , in
RecorderControlscomponent.The second change (see below) is directly related to this bug, but it will manifest in a similar defect with the
Stopbutton. Fixed preemptively.handleClick_RecordOnfunction,onStartRecordermust evaluate to a function (i.e. written asonStartRecorder()handleClick_RecordOfffunction,onStopRecordermust evaluate to a function (i.e. written asonStopRecorder()User Guide
No new behavior introduced
Expected behavior
User does see the allow screen share and select screen prompt
Screenshots
Additional context
None.
@soumik-mukherjee