Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

Major fix #241

Merged
merged 1 commit into from
Mar 12, 2013
Merged

Major fix #241

merged 1 commit into from
Mar 12, 2013

Conversation

avoidwork
Copy link
Owner

  • Handling 201s properly from client.response() by looking for the Location header or a URI in the response.responseText & making a subsequent GET to it which then resolves the original XHR request
  • Updating data.del() & data.set() to allow CORS requests to be resolved by the server (headers might not always be available)

…ocation` header or a URI in the `response.responseText` & making a subsequent `GET` to it which then resolves the original XHR request, updating `data.del()` & `data.set()` to allow CORS requests to be resolved by the server (headers might not always be available)

Forgot a `break`

Added braces for explicit meaning

Passing the wrong object when announcing 201 result
@ghost ghost assigned avoidwork Mar 12, 2013
avoidwork added a commit that referenced this pull request Mar 12, 2013
@avoidwork avoidwork merged commit 872569f into master Mar 12, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant