[WIP] [DO NOT MERGE] Update sdk to use loopback 3.x - #244
Conversation
|
@slnode test please |
| 'count', | ||
| 'prototype$updateAttributes', | ||
| 'prototype$patchAttributes', | ||
| 'updateAttributes', |
There was a problem hiding this comment.
in this test case it manually modifies the string so the test case passes....
is it a problem with https://github.com/strongloop/strong-remoting/blob/master/lib/rest-adapter.js has no concept of aliases? which is what sdk-angular interfaces with
not sure what the fix should be,
i see a couple options
- remap updateAttributes as main method, and use patchAttribute as alias instead
- prototype method aliases should get named the same way? (not sure about the current behavior need to dig deeper)
@bajtos thoughts?
There was a problem hiding this comment.
Because I see both updateOrCreate and upsert in the list above, I think both strong-remoting and this SDK support remote method aliases.
I would expect to see prototype$patchAttributes and prototype$updateAttributes listed for LB 3.0 (and hopefully for 2.x too).
|
closing in favor of #253 |
2 participants
tests are skipped because they broken, sdk ideally should be compatible with both Loopback 2 and 3, therefore please DO NOT MERGE