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

Add Flac to Samsung direct play profile #1030

Closed
wants to merge 2 commits into from

Conversation

ginjaninja1
Copy link

Add Flac to Samsung direct play profile.
Samsungs from 2012 support this..so if Flac support isnt universal for all Samsungs, it is most common...

Add Flac to Samsung direct play profile
@ginjaninja1
Copy link
Author

My first pull request, needs to be checked

@ginjaninja1 ginjaninja1 reopened this Feb 27, 2015
thanks to match the others it should have had new DirectPlayProfile.
Hopefully ive updated it
@ginjaninja1
Copy link
Author

its apparent im confused by github, i thought i had replicated the structure before...i edited it again to include your suggestion..but i cant see my correction :-(

@TheTedAdams
Copy link
Contributor

It's there now, your second commit showed up in this pull request. Click commits up top to see the second commit showed up, and then click files changed to see the final outcome of the 2 commits.

@ginjaninja1
Copy link
Author

thanks, its making a bit more sense...do i just wait now for an owner to accept or reject? is it submitted for review? do i need to close it to submit it?

@TheTedAdams
Copy link
Contributor

You are done. The Pull Request is submitted. One of the owners will review when they have time, and then they will either accept, reject, or ask for some changes. Might take a while, those guys have a lot on their plates.

@LukePulverenti
Copy link
Member

Does this work for all samsungs? what year did they start supporting flac?

@ginjaninja1
Copy link
Author

my 2012 model supports flac..i assume all later do as well ..but have not done any investigation..maybe a second samsung profile is a safer way to go..that said the number of unsupported tvs must be diminishing towards zero over time

@LukePulverenti
Copy link
Member

merged manually

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

3 participants