Skip to content

Commit

Permalink
remove id property from resource and testcase models
Browse files Browse the repository at this point in the history
its replced with _id - which is used in rest of models as well. ->consolidation..
  • Loading branch information
jupe committed Jan 29, 2019
1 parent d358b9e commit f4d332d
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
3 changes: 0 additions & 3 deletions app/models/resource.js
Original file line number Diff line number Diff line change
Expand Up @@ -210,9 +210,6 @@ const ResourceSchema = new Schema({
parent: {type: ObjectId, ref: 'Resource'}
});

ResourceSchema.virtual('id').get(function getId() {
return this._id;
});
ResourceSchema.set('toJSON', {
virtuals: true,
getters: true,
Expand Down
5 changes: 0 additions & 5 deletions app/models/testcase.js
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,6 @@ const TestCaseSchema = new Schema({
}
});

TestCaseSchema.virtual('id').get(function getId() {
return this._id;
});


TestCaseSchema.set('toJSON', {
virtuals: true,
getters: true,
Expand Down

0 comments on commit f4d332d

Please sign in to comment.