Skip to content

Conversation

@nitinvupk
Copy link

No description provided.

Copy link
Member

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 4
           

Complexity increasing per file
==============================
- plugins/myMetric/api/api.js  4
- plugins/myMetric/frontend/app.js  1
         

See the complete overview on Codacy

try {
params.qstring.args = JSON.parse(params.qstring.args);
} catch (SyntaxError) {
console.log('Parse ' + paramsInstance.qstring.args + ' JSON failed');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codacy found an issue: 'paramsInstance' is not defined.

}
});

let array = common.db.collection('my_metric').aggregate([
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codacy found an issue: 'common' is not defined.

console.log('Parse ' + paramsInstance.qstring.args + ' JSON failed');
}
}
common.db.collection('my_metric').insert(paramsInstance, function () {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codacy found an issue: 'common' is not defined.

}
let result = {};
validate(params, async function () {
common.db.collection("my_metric").find().sort({my_metric: -1}).limit(3).toArray(function (err, res) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codacy found an issue: 'common' is not defined.

@ar2rsawseen
Copy link
Member

Hello
no need for PR, just when you done, let our HR know with the link to your fork and we will review the task there :)

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.

2 participants