Skip to content

Changes to loopback lib/persisted_model.js to expose "upsertWithWhere" as REST API - #2496

Closed
mountain1234585 wants to merge 3 commits into
strongloop:masterfrom
mountain1234585:upsertWithWhereChanges
Closed

Changes to loopback lib/persisted_model.js to expose "upsertWithWhere" as REST API#2496
mountain1234585 wants to merge 3 commits into
strongloop:masterfrom
mountain1234585:upsertWithWhereChanges

Conversation

@mountain1234585

Copy link
Copy Markdown

Hi Raymond,

Kindly review the changes to the lib/persisted-model.js . I could see the new method appearing in the explorer and perform the operation.

Regards,
Sonali Samantaray

@slnode

slnode commented Jul 11, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

@slnode

slnode commented Jul 11, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

3 similar comments
@slnode

slnode commented Jul 11, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@slnode

slnode commented Jul 11, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@slnode

slnode commented Jul 11, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@mountain1234585

Copy link
Copy Markdown
Author

@raymondfeng
Can you please review this PR.

@slnode

slnode commented Jul 19, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

3 similar comments
@slnode

slnode commented Jul 19, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@slnode

slnode commented Jul 19, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@slnode

slnode commented Jul 19, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@mountain1234585

Copy link
Copy Markdown
Author

@Amir-61
cc @superkhau @bajtos

Can you pls review this PR as well ?

@mountain1234585

Copy link
Copy Markdown
Author

Have changes in another PR. #2539

Closing this, since the clahub was reporting errors even after adding all signatures and had too many conflicts to resolve.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants