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

bluetooth documentation #298

Merged
merged 1 commit into from
Apr 14, 2024
Merged

bluetooth documentation #298

merged 1 commit into from
Apr 14, 2024

Conversation

barrelltitor
Copy link
Contributor

Added the bluetooth documentation, let me know what should be changed in it!

@barrelltitor barrelltitor marked this pull request as ready for review April 14, 2024 10:53
@GioF71 GioF71 merged commit 95035fe into GioF71:main Apr 14, 2024
1 check passed
@GioF71
Copy link
Owner

GioF71 commented Apr 14, 2024

Hello, thank you for the great contribution.
There are not many changes I would do, I will ask you. Would it be ok if I open issues on your repo?
I can anticipate I would avoid to define the "default" card in ~/.asound.conf and then use aplay to the default card. I would instead create a named card and try to play to that with aplay.
Anyway we talk about very minor things.
Also markdown synthax but I can fix that myself.

Thank you again.

@barrelltitor
Copy link
Contributor Author

Just turned on issues on the fork. For testing yeah using aplay on a named card might be better to not forget it on the host, will look into that!

Thanks for fixing the markdown issues, I'm not very good with that yet

@barrelltitor barrelltitor deleted the patch-1 branch April 14, 2024 22:20
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.

None yet

2 participants