Update statistics to not include drafts #168
Conversation
kyle1morel
requested review from
wilwong89,
sanjaytkbabu and
qhanson55
as code owners
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
| ); | ||
| } | ||
|
|
||
| export async function down(knex: Knex): Promise<void> { |
There was a problem hiding this comment.
Similar blocks of code found in 6 locations. Consider refactoring.
| /* eslint-disable max-len */ | ||
| import type { Knex } from 'knex'; | ||
|
|
||
| export async function up(knex: Knex): Promise<void> { |
There was a problem hiding this comment.
Function up has 129 lines of code (exceeds 25 allowed). Consider refactoring.
| /* eslint-disable max-len */ | ||
| import type { Knex } from 'knex'; | ||
|
|
||
| export async function up(knex: Knex): Promise<void> { |
There was a problem hiding this comment.
Similar blocks of code found in 6 locations. Consider refactoring.
| ); | ||
| } | ||
|
|
||
| export async function down(knex: Knex): Promise<void> { |
There was a problem hiding this comment.
Function down has 129 lines of code (exceeds 25 allowed). Consider refactoring.
| @@ -0,0 +1,272 @@ | |||
| /* eslint-disable max-len */ | |||
There was a problem hiding this comment.
File 20240919000000_010-statistics-update.ts has 263 lines of code (exceeds 250 allowed). Consider refactoring.
|
Code Climate has analyzed commit b83f819 and detected 5 issues on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 0.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 36.7% (0.0% change). View more on Code Climate. |
Coverage Report (Frontend)
|
Coverage Report (Application)
|
Description
Update statistics function to not include submissions and enquiries in Draft status
Updates statistics table to clarify total count
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist
Further comments