Skip to content
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

Bugfix PR: Issue#1 microphone audio doesn't record along with screencast #11

Merged
merged 12 commits into from
Apr 16, 2020

Conversation

soumik-mukherjee
Copy link
Owner

Resolves #1

@soumik-mukherjee soumik-mukherjee added the bug Something isn't working label Apr 15, 2020
@soumik-mukherjee soumik-mukherjee added this to the Sprint 1 milestone Apr 15, 2020
@soumik-mukherjee soumik-mukherjee self-assigned this Apr 15, 2020
soumik-mukherjee and others added 7 commits April 15, 2020 22:03
* 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
@soumik-mukherjee
Copy link
Owner Author

@PassionateSolverSumit this is ready for review and approval 😌, let me know

@soumik-mukherjee
Copy link
Owner Author

Some recordings as test evidence:

  1. With audio recorded through my crappy headphone mic
    recording_1.zip

  2. With audio recorded through my crappy (even worse) laptop mic
    recording_2.zip

Things to note 🤓

  • There is a high speed fan running in the background, check out the noise suppression!
  • There is echo cancellation too! best way to test is enable feedback on the monitor (the embedded player that shows what's recording, you will find is on mute). Warning, unmute at your own risk, not recommended.

🤠

Copy link
Collaborator

@sumitkumar-Ray sumitkumar-Ray left a comment

Choose a reason for hiding this comment

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

left one test comment ..

README.md Show resolved Hide resolved
@soumik-mukherjee soumik-mukherjee merged commit f2d23b2 into master Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Microphone audio doesn't record along with screencast
2 participants