-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ch fix reading stats #104
Ch fix reading stats #104
Conversation
updatedAt: singleStat.updatedAt | ||
}; | ||
}) | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline before ')' function-paren-newline
@@ -483,7 +484,22 @@ class User { | |||
if (!stats || stats.length === 0) { | |||
return helper.jsonResponse(res, 404, { message: 'No article read' }); | |||
} | |||
return helper.jsonResponse(res, 200, { stats, statsCount: stats.length }); | |||
const allStats = await Promise.all( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
4757a7b
to
656378b
Compare
} | ||
} | ||
}); | ||
return helper.jsonResponse(res, 200, { statsArticles: allStats, thisWeekRead, thisMonthRead }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 524 exceeds the maximum line length of 100 max-len
updatedAt: singleStat.updatedAt | ||
}; | ||
}) | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline before ')' function-paren-newline
@@ -483,7 +485,43 @@ class User { | |||
if (!stats || stats.length === 0) { | |||
return helper.jsonResponse(res, 404, { message: 'No article read' }); | |||
} | |||
return helper.jsonResponse(res, 200, { stats, statsCount: stats.length }); | |||
const allStats = await Promise.all( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
What does this PR do?
Description of Task to be completed?
How should this be manually tested?
Any background context you want to provide?