You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This was reported internally on the ThinkNimble slack some time ago:
I ran into an issue with the TN models that Pari just helped me to solve. Just wanted to alert you, in case that means the code needs to be changed.
I was calling the List method, but it was conking out when mapping the results through FromAPI. I looked in the repo, and the code was
static fromAPI(json = {}) {
return new this(objectToCamelCase(json))
}
when we changed it to
static fromAPI(json = {}) {
return new User(objectToCamelCase(json))
}
This should be investigated - this in the static fromAPI method provided by tn-models should refer to the enclosing class, but for some reason was not in this case.
The text was updated successfully, but these errors were encountered:
This was reported internally on the ThinkNimble slack some time ago:
I ran into an issue with the TN models that Pari just helped me to solve. Just wanted to alert you, in case that means the code needs to be changed.
I was calling the List method, but it was conking out when mapping the results through FromAPI. I looked in the repo, and the code was
when we changed it to
This should be investigated -
this
in thestatic fromAPI
method provided by tn-models should refer to the enclosing class, but for some reason was not in this case.The text was updated successfully, but these errors were encountered: