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

Fix teams ignoring input sources e.g. microphones #4779

Merged
merged 2 commits into from
Dec 18, 2021

Conversation

seonwoolee
Copy link

teams-for-linux.profile and teams.profile both include electron.profile, which has the noinput option, which blocks microphones and other input sources, so this option needs to be ignored.

Copy link
Collaborator

@rusty-snake rusty-snake left a comment

Choose a reason for hiding this comment

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

Don't put them in the "Disabled until someone reported positive feedback" block.


Is this special for teams or does it also happen with e.g. firefox?

@seonwoolee
Copy link
Author

Sorry, I was trying to respect the ordering rules for pull requests.

It's not a general problem, just any electron app that needs microphone input. Not applicable to Firefox

@@ -13,6 +13,8 @@ ignore include whitelist-usr-share-common.inc
ignore novideo
ignore private-tmp

ignore novideo
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you meant ignore noinput.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Still unresolved.

@kmk3
Copy link
Collaborator

kmk3 commented Dec 14, 2021

Also, could you squash the commits and push? Example:

git stash; git tag backup # just in case
git rebase -i "$(git merge-base master HEAD)..HEAD"
# Change all occurrences of "pick" to "fixup", except for the first one
# (This makes the first commit "absorb" all the other commits)
# Check if it looks good (if not, `git reset --hard backup` and try again)
# Then just push
git push --force-with-lease

@netblue30 netblue30 merged commit d30402f into netblue30:master Dec 18, 2021
@netblue30
Copy link
Owner

merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants