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

Allow specification of media type #2398

Merged

Conversation

Projects
None yet
3 participants
@NAJ10
Copy link

NAJ10 commented Feb 9, 2019

fixes #2397

NAJ10
@SystemFw
Copy link
Contributor

SystemFw left a comment

Test failure seems unrelated

jsonDecoder[F].flatMapR { json =>
jsonOf(MediaType.application.json)

def jsonOf[F[_]: Sync, A](r1: MediaRange, rs: MediaRange*)(

This comment has been minimized.

Copy link
@ChristopherDavenport

ChristopherDavenport Feb 11, 2019

Member

This introduces an overload. Could we get this version renamed?

NAJ10

@ChristopherDavenport ChristopherDavenport merged commit bc76769 into http4s:master Feb 12, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@NAJ10 NAJ10 deleted the NAJ10:http4s-issue-2397-override-mediatype branch Feb 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.