Skip to content
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

BHBC-1814: Promote Dev to Test #796

Merged
merged 24 commits into from
Jun 21, 2022
Merged

BHBC-1814: Promote Dev to Test #796

merged 24 commits into from
Jun 21, 2022

Conversation

NickPhura
Copy link
Collaborator

Overview

Links to Jira tickets

  • {List all applicable Jira tickets}

Description of relevant changes

  • {List all relevant changes, in particular anything that will help the reviewers test/verify this PR}

PR Checklist

A list of items that are good to consider when making any changes.

Note: this list is not exhaustive, and not all items are always applicable.

Code

  • New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
  • I have avoided duplicating code when possible, moving re-usable pieces into functions
  • I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
  • My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
  • My code changes account for null/undefined values and handle errors appropriately
  • My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

  • My code follows the established style conventions
  • My code uses native material-ui components/icons/conventions when possible

Documentation

  • I have commented my code sufficiently, such that an unfamiliar developer could understand my code
  • I have added/updated README's and related documentation, as needed

Tests

  • I have added/updated unit tests for any code I've added/updated
  • I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

  • I have run the linter and fixed any issues, as needed
    See the lint commands in package.json
  • I have run the formatter and fixed any issues, as needed
    See the format commands in package.json

SonarCloud

  • I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

NickPhura and others added 24 commits May 9, 2022 12:36
… Restoration, Adjust access request approval to account for existing bceid-basic-business identity sources (#771)

- Port improved user access request functionality from Restoration
  - Better/fixed handling for approving/denying user requests
  - Adjust access request approval to account for existing bceid-basic-business identity sources
- Existing access requests created before we accounted for bceid-basic-and-business identity sources should no longer throw an error when trying to approve/deny them.
…er (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.
* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive
- account for the fact that openapi enums are case sensitive
* Create toDraft.yml

* remove dependency
Bumps [sqlite3](https://github.com/TryGhost/node-sqlite3) from 5.0.2 to 5.0.8.
- [Release notes](https://github.com/TryGhost/node-sqlite3/releases)
- [Changelog](https://github.com/TryGhost/node-sqlite3/blob/master/CHANGELOG.md)
- [Commits](TryGhost/node-sqlite3@v5.0.2...v5.0.8)

---
updated-dependencies:
- dependency-name: sqlite3
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Nick Phura <[email protected]>
* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum
- Fix cypress tests
- Update n8n version/config
* BHBC-1755: Add backbone api env var and platform-service.
- Remove clamav hardcoded default values
- Improve checking for required clamav env vars before initializing scanner
- Add missing clamav env vars to api deployment yaml
Bumps [convict](https://github.com/mozilla/node-convict) from 6.2.2 to 6.2.3.
- [Release notes](https://github.com/mozilla/node-convict/releases)
- [Changelog](https://github.com/mozilla/node-convict/blob/master/CHANGELOG.md)
- [Commits](mozilla/node-convict@v6.2.2...v6.2.3)

---
updated-dependencies:
- dependency-name: convict
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Fixed an error that occurred when attempting to edit the ‘Purpose and Methodology’ section of a survey.

 - Updated schema validation for GET on purpose_methodology update endpoint.
 - BHBC-1783: Add API response validation testing to update endpoint (work in progress).
 - BHBC-1783: Amend platform-service and its test file w 'dwc/submission/create' endpoint.
…Dev (#793)

 - Replace hardcoded URLs in Resources page by implementing env. vars (BHBC-1775)
 - Fix API response validation bugs for Project Drafts (BHBC-1809)
BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift
Tech debt: survey-service
EML: major updates to EML creation code
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov
Copy link

codecov bot commented Jun 21, 2022

Codecov Report

Merging #796 (1e74bf6) into test (b99f8cb) will decrease coverage by 2.25%.
The diff coverage is 83.80%.

@@            Coverage Diff             @@
##             test     #796      +/-   ##
==========================================
- Coverage   73.05%   70.80%   -2.26%     
==========================================
  Files         353      367      +14     
  Lines       12905    12472     -433     
  Branches     2317     2288      -29     
==========================================
- Hits         9428     8831     -597     
- Misses       2994     3158     +164     
  Partials      483      483              
Impacted Files Coverage Δ
api/src/__mocks__/db.ts 92.59% <0.00%> (-7.41%) ⬇️
api/src/app.ts 0.00% <0.00%> (ø)
api/src/constants/database.ts 100.00% <ø> (ø)
api/src/openapi/schemas/administrative-activity.ts 100.00% <ø> (ø)
api/src/openapi/schemas/draft.ts 100.00% <ø> (ø)
api/src/openapi/schemas/permit-no-sampling.ts 100.00% <ø> (ø)
api/src/paths/codes.ts 100.00% <ø> (ø)
api/src/paths/dwc/scrape-occurrences.ts 0.00% <ø> (ø)
api/src/paths/dwc/view-occurrences.ts 96.66% <ø> (ø)
.../src/paths/project/{projectId}/attachments/list.ts 96.87% <ø> (ø)
... and 266 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d7e0fa...1e74bf6. Read the comment docs.

@NickPhura NickPhura merged commit 71c34d1 into test Jun 21, 2022
@NickPhura NickPhura changed the title Promote Dev to Test BHBC-1814: Promote Dev to Test Jun 22, 2022
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.

4 participants