Skip to content

BCMOHAD-29760_JS_upgrade #1144

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.

Sign up for GitHub

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

Merged
chloe-yuu merged 41 commits into from
Dec 3, 2025

Merge pull request #1145 from bcgov/BCMOHAM-28761_Bitnami_MongoDB

e8e972d
Select commit
Loading
Failed to load commit list.
Merged

BCMOHAD-29760_JS_upgrade #1144

Merge pull request #1145 from bcgov/BCMOHAM-28761_Bitnami_MongoDB
e8e972d
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Nov 20, 2025 in 1m 8s

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 16 in client/src/pages/private/CohortHeader.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'cohort.cohort_name' is missing in props validation 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwnzO0rq0RYeTY_X&open=AZnvDwnzO0rq0RYeTY_X&pullRequest=1144

Check warning on line 394 in server/keycloak.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

arrow function is equivalent to `Boolean`. Use `Boolean` directly. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw6GO0rq0RYeTZB9&open=AZnvDw6GO0rq0RYeTZB9&pullRequest=1144

Check warning on line 85 in client/src/components/generic/Header.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwPYO0rq0RYeTY-C&open=AZnvDwPYO0rq0RYeTY-C&pullRequest=1144

Check warning on line 7 in client/src/pages/public/Keycloak.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The arrow function should be named. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwrqO0rq0RYeTY_r&open=AZnvDwrqO0rq0RYeTY_r&pullRequest=1144

Check warning on line 3 in server/scripts/parse-xml.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `node:path` over `path`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw2mO0rq0RYeTZBJ&open=AZnvDw2mO0rq0RYeTZBJ&pullRequest=1144

Check warning on line 236 in server/scripts/randomize-participants-status.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw3HO0rq0RYeTZBc&open=AZnvDw3HO0rq0RYeTZBc&pullRequest=1144

Check warning on line 232 in client/src/pages/private/ParticipantEOI.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwnWO0rq0RYeTY_U&open=AZnvDwnWO0rq0RYeTY_U&pullRequest=1144

Check failure on line 106 in client/src/components/modal-forms/IndigenousDeclarationForm.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwa9O0rq0RYeTY-d&open=AZnvDwa9O0rq0RYeTY-d&pullRequest=1144

Check warning on line 346 in server/services/cohorts.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use an object literal as default. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw0nO0rq0RYeTZA7&open=AZnvDw0nO0rq0RYeTZA7&pullRequest=1144

Check warning on line 65 in client/src/components/employer-form/OrgBookSearch.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwc4O0rq0RYeTY-g&open=AZnvDwc4O0rq0RYeTY-g&pullRequest=1144

Check warning on line 72 in client/src/components/participant-form/EmailSubmissionForm.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Pass a message to the `Error` constructor. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwUDO0rq0RYeTY-T&open=AZnvDwUDO0rq0RYeTY-T&pullRequest=1144

Check warning on line 200 in client/src/components/participant-form/Fields.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwTjO0rq0RYeTY-S&open=AZnvDwTjO0rq0RYeTY-S&pullRequest=1144

Check warning on line 137 in server/scripts/parse-xml.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw2mO0rq0RYeTZBT&open=AZnvDw2mO0rq0RYeTZBT&pullRequest=1144

Check warning on line 33 in server/logger.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `export…from` to re-export `default`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw4uO0rq0RYeTZB0&open=AZnvDw4uO0rq0RYeTZB0&pullRequest=1144

Check warning on line 320 in client/src/components/generic/Table.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `new Array()` instead of `Array()`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwRxO0rq0RYeTY-P&open=AZnvDwRxO0rq0RYeTY-P&pullRequest=1144

Check warning on line 67 in server/scripts/randomize-participants.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer top-level await over an async IIFE. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw4GO0rq0RYeTZBv&open=AZnvDw4GO0rq0RYeTZBv&pullRequest=1144

Check warning on line 29 in client/src/pages/private/PSIViewDetails.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The arrow function should be named. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwq3O0rq0RYeTY_o&open=AZnvDwq3O0rq0RYeTY_o&pullRequest=1144

Check warning on line 21 in client/src/pages/private/ParticipantActionSuccess.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwhsO0rq0RYeTY-7&open=AZnvDwhsO0rq0RYeTY-7&pullRequest=1144

Check warning on line 39 in client/src/routes/index.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwtzO0rq0RYeTY_8&open=AZnvDwtzO0rq0RYeTY_8&pullRequest=1144

Check warning on line 45 in client/src/pages/private/SiteViewDetailsTabs.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The arrow function should be named. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwlNO0rq0RYeTY_J&open=AZnvDwlNO0rq0RYeTY_J&pullRequest=1144

Check warning on line 53 in server/tests/util/keycloak.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwwDO0rq0RYeTZAX&open=AZnvDwwDO0rq0RYeTZAX&pullRequest=1144

Check warning on line 3 in server/scripts/services/participant-seed.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `node:path` over `path`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDw2NO0rq0RYeTZBG&open=AZnvDw2NO0rq0RYeTZBG&pullRequest=1144

Check warning on line 58 in client/src/routes/index.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwtzO0rq0RYeTZAA&open=AZnvDwtzO0rq0RYeTZAA&pullRequest=1144

Check warning on line 20 in client/src/index.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis.window` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwuyO0rq0RYeTZAI&open=AZnvDwuyO0rq0RYeTZAI&pullRequest=1144

Check warning on line 14 in client/src/components/generic/Page.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=bcgov_hcap&issues=AZnvDwPFO0rq0RYeTY-B&open=AZnvDwPFO0rq0RYeTY-B&pullRequest=1144
View more details on SonarQubeCloud