Provide way to set both status code and status text in renderData() #385

Closed
seancorfield opened this Issue Oct 21, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@seancorfield
Member

seancorfield commented Oct 21, 2015

Currently you can only specify the status code in renderData() but it would be useful to be able to specify status text as well sometimes. Since the JSONP callback is the optional 4th argument, probably the easiest way to support this is to allow status code to be either a numeric value on its own (as now) or a vector pair of status code and text.

@seancorfield seancorfield self-assigned this Oct 21, 2015

@seancorfield seancorfield added this to the 3.6 milestone Oct 21, 2015

@seancorfield

This comment has been minimized.

Show comment
Hide comment
@seancorfield

seancorfield Oct 22, 2015

Member

The nicest way to support this would be via #388 but that's pending input from the community.

Member

seancorfield commented Oct 22, 2015

The nicest way to support this would be via #388 but that's pending input from the community.

@seancorfield

This comment has been minimized.

Show comment
Hide comment
@seancorfield

seancorfield Oct 31, 2015

Member

Note: setStatus( int, String ) is actually deprecated but we do not want to use sendError( int, String ) here.

Member

seancorfield commented Oct 31, 2015

Note: setStatus( int, String ) is actually deprecated but we do not want to use sendError( int, String ) here.

seancorfield added a commit that referenced this issue Nov 4, 2015

Rollback sendError() change in #385
As I noted in the comments, using `sendError()` is not what we want
anyway. I should have listened to myself :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment