Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Commit e83e3b8

Browse files
author
sachin-maheshwari
authored
Merge pull request #123 from topcoder-platform/feature/shapeup4-cqrs-update2
DELETE fix : Shapeup4 CQRS Update
2 parents 99d8b0a + a19661c commit e83e3b8

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Diff for: src/common/service-helper.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ async function deleteChild (model, id, params, resourceName, transaction) {
218218
params.forEach(attr => { esParams[attr] = record[attr] })
219219

220220
// remove from db
221-
await dbHelper.remove(model, record.id, transaction)
221+
await dbHelper.remove(model, record.id, null, transaction)
222222
await deleteRecordFromEs(record.id, esParams, resourceName)
223223

224224
// sleep for configured time

Diff for: src/modules/user/service.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ search.schema = {
175175
* @return {Promise<void>} no data returned
176176
*/
177177
async function remove (id, auth, params) {
178-
beginCascadeDelete(id, params)
178+
await beginCascadeDelete(id, params)
179179
}
180180

181181
/**

0 commit comments

Comments
 (0)