Added test to issue #560 #561

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
@ghost

ghost commented Nov 19, 2013

@justinbmeyer

View changes

model/model_test.js
+ var Person = can.Model("Person",{
+ update : function(id, attrs, success, error){
+ return can.ajax({
+ url : "/people/"+id,

This comment has been minimized.

@justinbmeyer

justinbmeyer Nov 20, 2013

Contributor

Can you change this to not use fixture: function() ... this has been deprecated for a long time. Instead, do:

can.fixture("POST /people/{id}",function(){
  return  {createdAt: "now"}
})

Thanks!

@justinbmeyer

justinbmeyer Nov 20, 2013

Contributor

Can you change this to not use fixture: function() ... this has been deprecated for a long time. Instead, do:

can.fixture("POST /people/{id}",function(){
  return  {createdAt: "now"}
})

Thanks!

@ccummings

This comment has been minimized.

Show comment
Hide comment
@ccummings

ccummings Apr 25, 2014

Contributor

#560 is fixed and an accompanying test was included in the fix. Closing

Contributor

ccummings commented Apr 25, 2014

#560 is fixed and an accompanying test was included in the fix. Closing

@ccummings ccummings closed this Apr 25, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment