Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ManognaMutyala
Copy link
Contributor

@ManognaMutyala ManognaMutyala commented Oct 10, 2017

Allowing user to explore more HTTP Responses

Pull Request Checklist

Helpful things

Provides detailed description about various methods that can be used by user.

Minor fix

Purpose

It is done in order to provide user better understandibility

What does this PR do?
Provides description of various Result methods for the user.

Why did you take this approach?
To provide detailed information

Allowing user to explore more HTTP Responses
@[simple-action](code/javaguide/http/JavaActions.java)

An action returns a `play.mvc.Result` value, representing the HTTP response to send to the web client. In this example `ok` constructs a **200 OK** response containing a **text/plain** response body.
An action returns a `play.mvc.Result` value, representing the HTTP response to send to the web client. In this example `ok` constructs a **200 OK** response containing a **text/plain** response body.For more examples of HTTP response check out [HTTP Responses] (https://www.playframework.com/documentation/2.5.8/api/java/play/mvc/Results.html)
Copy link
Member

Choose a reason for hiding this comment

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

This is the 2.6.x docs referencing 2.5.8 javadocs. You should instead link to /api/java/play/mvc/Results.html. So you can rewrite it to:

For more examples of HTTP responses see [`play.mvc.Result` methods](/api/java/play/mvc/Results.html#method.summary).

@ManognaMutyala
Copy link
Contributor Author

ManognaMutyala commented Oct 11, 2017

Thanks for the info sir and I have made the changes as you suggested.

@marcospereira marcospereira merged commit 06f9804 into playframework:2.6.x Nov 24, 2017
marcospereira pushed a commit that referenced this pull request Nov 24, 2017
Allowing user to explore more HTTP Responses
@marcospereira
Copy link
Member

Forward port to master: 50a2068

@marcospereira marcospereira added this to the Play 2.6.8 milestone Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants