Code review comment for lp:~xavi-garcia-mena/indicator-sound/show-microphone-volume-external-mic

Revision history for this message
Xavi Garcia (xavi-garcia-mena) wrote :

Thanks for the review, Charles.
I've added that function.

I'm already working on tests...

I need to find out how to add some pulseaudio logic in order to test sources with certain properties.

We'll need also to add sinks using the microphone, simulating a fake client recording audio...

As we are already testing sink outputs I don't think sinks are going to be an issue.

I hope to finish that soon.

« Back to merge proposal