fix(core): reduce the updateById 3 DB calls to 2 (#36)

* fix(core): reduce the updateById 3 DB calls to 2

GH-33

* docs(ci-cd): added PR template

GH-33

Author: yeshamavani

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,30 @@
+## Description
+
+Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
+
+Fixes # (issue)
+
+## Type of change
+
+Please delete options that are not relevant.
+
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
+- [ ] Intermediate change (work in progress)
+
+## How Has This Been Tested?
+
+Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
+
+- [ ] Test A
+- [ ] Test B
+
+## Checklist:
+
+- [ ] Performed a self-review of my own code
+- [ ] npm test passes on your machine
+- [ ] New tests added or existing tests modified to cover all changes
+- [ ] Code conforms with the style guide
+- [ ] API Documentation in code was updated
+- [ ] Any dependent changes have been merged and published in downstream modules

package-lock.json

@@ -6,7 +6,7 @@ import {
   EntityCrudRepository,
   Where,
 } from '@loopback/repository';
-import {keyBy} from 'lodash';
+import {keyBy, cloneDeep} from 'lodash';
 
 import {Action, AuditLog} from '../models';
 import {AuditLogRepository} from '../repositories';
@@ -202,7 +202,8 @@ export function AuditRepositoryMixin<
         options = {noAudit: true};
       }
       await super.updateById(id, data, options);
-      const after = await this.findById(id);
+      let after = cloneDeep(before);
+      after = Object.assign(after, data);
 
       if (this.getCurrentUser) {
         const user = await this.getCurrentUser();