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-1736: Fix Read More/Read Less idempotence bug #772

Merged
merged 5 commits into from
May 9, 2022
Merged

Conversation

curtisupshall
Copy link
Contributor

Overview

Links to Jira tickets

https://quartech.atlassian.net/browse/BHBC-1736

Description of relevant changes

Tightens qualification for string truncating in the ReadMoreField component to avoid an issue where pressing Read More/Read Less results in no change.

@codecov
Copy link

codecov bot commented May 9, 2022

Codecov Report

Merging #772 (5704337) into dev (c57bbc6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #772   +/-   ##
=======================================
  Coverage   71.96%   71.97%           
=======================================
  Files         364      364           
  Lines       12706    12708    +2     
  Branches     2356     2357    +1     
=======================================
+ Hits         9144     9146    +2     
  Misses       3037     3037           
  Partials      525      525           
Impacted Files Coverage Δ
app/src/components/fields/ReadMoreField.tsx 100.00% <100.00%> (ø)

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 c57bbc6...5704337. Read the comment docs.

anissa-agahchen
anissa-agahchen previously approved these changes May 9, 2022
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 9, 2022

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
0.0% 0.0% Duplication

@curtisupshall curtisupshall merged commit 8639bc1 into dev May 9, 2022
@NickPhura NickPhura deleted the BHBC-1736 branch May 9, 2022 21:28
NickPhura added a commit that referenced this pull request May 10, 2022
* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

Co-authored-by: Curtis Upshall <[email protected]>
NickPhura added a commit that referenced this pull request May 10, 2022
* Update README.md

* First commit (#566)

* First commit

* Upload

* Update the sec produre for unseceruing an attachment

* Version update

* Update for single upload button

* New fixtures

* Update

* Survey script

* delete survey

* Spacing

* Moving to test (#611)

* Update README.md

* First commit (#566)

* First commit

* Upload

* Update the sec produre for unseceruing an attachment

* Version update

* Update for single upload button

* New fixtures

* Update

* Survey script

* delete survey

* Spacing

* Update access request page: remove regional offices section, remove non-supported roles from dropdown (#610)

* [BHBC-1449] Add Help Link (#614)

* Update template validation schemas (#612)

* Add migration to update moose SRB/Composition Validation schema
* Add Summary Statistics file to resources page
* Add Date column to recruitment survey
* Fix dependency warnings
* Remove unused test function

* Update deployStatic.yml

* Move into test (#615)

* Update README.md

* First commit (#566)

* First commit

* Upload

* Update the sec produre for unseceruing an attachment

* Version update

* Update for single upload button

* New fixtures

* Update

* Survey script

* delete survey

* Spacing

* Update access request page: remove regional offices section, remove non-supported roles from dropdown (#610)

* [BHBC-1449] Add Help Link (#614)

* Update template validation schemas (#612)

* Add migration to update moose SRB/Composition Validation schema
* Add Summary Statistics file to resources page
* Add Date column to recruitment survey
* Fix dependency warnings
* Remove unused test function

* Update deployStatic.yml

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: jeznorth <[email protected]>

* BHBC-1363 (#599)

Support project/survey report attachments with meta

* Add admin only endpoint to update the api log level at runtime (#619)

* Add admin only endpoint to update the api log level at runtime

* Update test

* Adding Test scripts (#617)

* Adding Test scripts

* Create project and Survey

* Remove second test file

* Test Updates/Fixes (#620)

Test updates/Fixes

* BHBC-1363-2: Fix signed url for reports, update fileupload to support single file only settings. (#621)

* BHBC-1461 (#622)

* Readme makefile updates (#623)

* Update Makefile, Readme, env
* Rename BioHub -> SIMS

* Add `Order By` clause to species sql (#627)

* BHBC-1462: Add new record to proprietary_type lookup table. (#629)

- Update code to leverage the `is_first_nation` column from the proprietary_type table.
  - Previously the code was using the row id

* BHBC-1470: View/edit survey and project metadata (#624)

BHBC-1470: View/edit survey and project metadata (#624)

* BHBC-1478: Update API to check project level permissions. (#630)

* Combine dependabot (#634)

* Bump tmpl from 1.0.4 to 1.0.5 in /app

Bumps [tmpl](https://github.com/daaku/nodejs-tmpl) from 1.0.4 to 1.0.5.
- [Release notes](https://github.com/daaku/nodejs-tmpl/releases)
- [Commits](https://github.com/daaku/nodejs-tmpl/commits/v1.0.5)

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

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

* Bump ws from 5.2.2 to 5.2.3 in /app

Bumps [ws](https://github.com/websockets/ws) from 5.2.2 to 5.2.3.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@5.2.2...5.2.3)

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

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

* Bump validator from 13.6.0 to 13.7.0 in /n8n

Bumps [validator](https://github.com/validatorjs/validator.js) from 13.6.0 to 13.7.0.
- [Release notes](https://github.com/validatorjs/validator.js/releases)
- [Changelog](https://github.com/validatorjs/validator.js/blob/master/CHANGELOG.md)
- [Commits](validatorjs/validator.js@13.6.0...13.7.0)

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

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

* Update

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* migration (#637)

* Bhbc 1470 (#638)

* Small test script and seeding changes (#631)

* Small test script and seeding changes

* Remove Shreyas from the seed

Co-authored-by: Nick Phura <[email protected]>

* Create zap.yml

* Update zap.yml

* Create dev.context

* Update zap.yml

* Zap follow up (#640)

* Small test script and seeding changes

* Remove Shreyas from the seed

* Details for ZAP

* Subtitute hardcoded PW with GHA- Secret

* [BHBC-1470] Increase Report Summary Input Size (#641)

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* BHBC-1467: One page project/survey layout (#636)

One page project/survey layout

* Test Script update, login and some other details (#646)

* Test Script update, login and some other details

* Add delay to login

* Remove file

* Pointing to the right authurl

* Path confusion

* And again

* It is all in the timing

* Update readme as per Nick's comment

* BHBC-950 (#643)

- System admin users can delete other system users.

* Update DialogContext.Snackbar(#647)

Update snackbar to accept ReactNode as message

* Update deployStatic.yml

* [BHBC-950] UI Cleanup (#648)

* [BHBC-950] UI Cleanup

* [BHBC-950] Updating UI

* Trigger Build

* Update to test script to deal with changed id for report upload

* [BHBC-950] Fixing Request Access Table

Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: Anissa Agahchen <[email protected]>

* Update deployStatic.yml

* BHBC-949: Manage Project Team - Add + Delete User (#645)

- Add project users page
- Add project users "add project participant" form
- Add delete project user functionality
- Update some api project role authorization checks
- Update some frontend role check functionality
- misc fixes/updates

* Update deployStatic.yml

* First test (#650)

* First test

* Update path

* - update older node versions to 14
- Increase tsconfig versions
- Increase a few package.json versions
- Update a few git actions to latest versions

* Revert accidental backup file change

* Set back to es2015 otherwise the test framework will fail

Co-authored-by: Nick Phura <[email protected]>

* Update deployStatic.yml

* Manage system/project roles (#649)

* BHBC-948: Allow users to change existing system user roles.
* BHBC-1491: Allow users to change existing project participant's roles.

* Update deployStatic.yml

* [BHBC-948] UI Updates / Cleanup (#652)

* Updates to Agancies and Proeject Coordinator name/label

* Revert "Merge branch 'dev' of https://github.com/bcgov/biohubbc into dev"

This reverts commit 16b5f48, reversing
changes made to 1c88d34.

* Changes to codes.ts for removing BC and all over for the coordinator vs contact name … (#654)

* Cahnges to codes.ts and all over for the coordinator ro contact name change

* Update snapshots

* Finish BHBC-1505

* Update deployStatic.yml

* BHBC-1457 (#651)

Implemented EML endpoint and addition to target DwC archive file on S3. Moved all EML creation functionality to middle tier tooling.

* Tech debt (#653)

* Split up the workflow for uploading project/survey attachments and project/survey reports
* associated tests

* Updates to the test script and table changes (#658)

* All changes

* Update to formatting

* Remove new table

* Bhbc 1499 (#656)

Updates to the Moose template and transformation configs

* BHBC-1527: system and project roles updated (#659)

* update url for download (#660)

* Fix wonky survey view/view-for-update queries + misc (#664)

* Fix survey view/view-for-update queries + misc

* BHBC-1496: Transform Lat/Long to Darwin Core Accepted Format (#667)

- Enhance transformations to allow multiple source columns in parse step (uses first non-null value)
- Update scrape SQL to account for UTM or LatLong verbatimCoordinates
- Add `parseLatLongString` spatial-util function to parse a lat long string.

* BHBC-1525 (#669)

* BHBC-1525
- initial EML test file and low hanging target testing

* remove unused system accounts (#666)

* remove unused system accounts
* update code to reference previously updated role names

* Security rules for PorH (#668)

* Security rules for PorH

* Endpoint Tests (#661)

* Add missing `connection.commit`'s (#671)

* Update endpoint role authorization (#670)

* Update endpoint role authorization
* Update occurrence scrape verbatimCoordinates handling

* Bhbc 1400 (#657)

Goat template validations and transformations
Sheep template validations and transformations

* UI Updates / Clean-up (#674)

* Update all the BC and DC to tune the resource usage (#678)

* BHBC-1523: Add new users (#675)

* Added "+ new users" button on Admin user page and added testing for both API and app.

* Bhbc 1529- Add custom props to templates (#677)

* BHBC-1530: Update Moose transformations to account for all fields. (#680)

* BHBC-1530
- Enhance xlsx transformation parse step
- Update Moose validation/transformation schemas
- Minor refactor to part of transformation code
- Update resource page link

* BHBC-1520: View user details (#679)

- Ability to add system users from the manage users page
  - Associated endpoints and tests
- Page to view user details and their associated projects
  - Ability to remove user from projects or update their role.
  - Associated endpoints and tests
- Reduce some api code duplication

* BHBC-1531 (#683)

* BHBC-1531
    - adjusted taxonomic codes to exclude those codes that have end date setc
* Fix focal species yup schema validation rule

* BHBC-1520 (#684)

Prevent projects from having no Project Lead role: bug fixes + tests

* BHBC-1512: Added zoom for fullscreen map on project details. (#685)

Added zoom for fullscreen map on project details

* Enhance useInterval hook to support a timeout. Add unit tests. (#686)

* BHBC-1512-2: Additional map scroll wheel zoom changes (#687)

* BHBC-1512-2: Additional map scroll wheel zoom changes

* BHBC-1545: Tech Debt: Re-export all queries via queries.ts. Update custom error naming. (#688)

Add index files for all query folders
Add queries.ts to re-export all queries.
Update all references to existing queries to instead reference queries.ts
Rename `CustomError.ts->CustomError` to `custom-error.ts->HTTPError` and all references.
Run formatter/linter.

* BHBC-1545: Tech Debt: Updated apidoc for /user/* (#681)

Update openapi doc for /user/* endpoints
Misc unit test updates

* BHBC-1503: GCNotfiy (#689)

* Created GCNotify Api endpoint for generic email functionality

* BHBC-1557: Tech Debt 3 (#690)

* Add user-service.ts
Move user functions to user-service.ts.
Update tests.

Add code-service.ts
Move code functions to code-service.ts
Update tests.

Misc root-api-doc.ts cleanup

Add swagger-ui container to docker-compose

* Add SwaggerUI to docker-compose.yml (for local dev currently)

* Minor makefile updates

* Updates to error classes

* Updates to error logging

* Minor test tweak

* Handle DatabaseErrors

* remove console logs

* BHBC-1492: Email Access Request (#692)

* BHBC-1557: Tech Debt - Update App route handling and route security. (#693)

* Tech debt: Fix partnerships, and update /project/{projectId}/update endpoint (#694)

* BHBC-1557: Ke-tech-debt (#698)

* updated openApi schemas

* Added Survey creation. (#700)

* Unify dependabot (#701)

* Bump nanoid from 3.1.30 to 3.2.0 in /n8n (#704)

* Bump copy-props from 2.0.4 to 2.0.5 in /api (#702)

* Bump node-fetch from 2.6.5 to 2.6.7 in /n8n (#705)

* BHBC-1557: Add ProjectService. (#699)

* Bump lodash from 4.17.20 to 4.17.21 in /app (#703)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.20 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.20...4.17.21)

---
updated-dependencies:
- dependency-name: lodash
  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: Anissa Agahchen <[email protected]>

* BHBC-1493: Approval Email For Access Request  (#708)

Approval email sent to idir email connected to access request user.

* Dev ops  scale schemaspy up and down (#709)

* Scale SchemaSpy as part of the pipeline to make sure it is current all the time

* Make it a one step SchemaSpy cycle

* Formatting

* Fix typo

* Reduce cpu/request limits

* Revert "Reduce cpu/request limits"

This reverts commit b99f8cb.

* Reduce cpu/request limits + Adjust default Log Level (#716)

* BCEID bug fix + improvements (#717)

* BHBC-1613 - support bceid login (#720)

* BHBC-1580: Updates to moose/sheep/goat templates (#725)

* Tech debt 6 (#707)

* Disable html logging when running app tests.
* Update logger utils, make mocha disable winston logger before test execution
* Add missing type definition for @tmcw/togeojson
* Updates to map components (from resto)
* Update typescript and knex versions, remove swagger ui docker compose step, add swagger-ui directly to app
* Incorporate dependabot changes
* Update eslint config
* Run lint-fix using updated eslint rules and fix issues
* Update startend date component
* Move old `/users` endpoint to `/user/list`

* First cut of the postman collection with automatic login (#726)

* Isolating Service Code (#724)

Port changes from restoration to move functionality to services

* Update siteminder url (#736)

* Survey- Additional Fields (#731)

* create/view/edit  survey page with new purpose and methodology form
* addition of "ecological season", "intended outcome" and "vantage" lookup tables
& removal of common survey methodology - field table name change to "field method"
* "survey.objectives" attribute name change to "survey.additional_details"
*  seeding new lookup tables
* re-seed "field method" table
* tests

* package lock updates for minimist (#740)

* dependabot change version for moment (#739)

* Bump convict from 6.2.0 to 6.2.2 in /n8n (#745)

Bumps [convict](https://github.com/mozilla/node-convict) from 6.2.0 to 6.2.2.
- [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.0...v6.2.2)

---
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>

* Bump async from 2.6.3 to 2.6.4 in /app (#743)

Bumps [async](https://github.com/caolan/async) from 2.6.3 to 2.6.4.
- [Release notes](https://github.com/caolan/async/releases)
- [Changelog](https://github.com/caolan/async/blob/v2.6.4/CHANGELOG.md)
- [Commits](caolan/async@v2.6.3...v2.6.4)

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

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump async from 3.2.0 to 3.2.3 in /testing/e2e (#742)

Bumps [async](https://github.com/caolan/async) from 3.2.0 to 3.2.3.
- [Release notes](https://github.com/caolan/async/releases)
- [Changelog](https://github.com/caolan/async/blob/master/CHANGELOG.md)
- [Commits](caolan/async@v3.2.0...v3.2.3)

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

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* BHBC-1688: Add survey radio field (#746)

* BHBC-1688: Add new radio button field to survey.
* Add github action to write openshift urls

* Techdebt (#741)

remove default logs
fix tests
fix code smells

* Misc Cleanup (#748)

* misc cleanup
* Update user seed file

* BHBC-1690: Add scroll to error formik component (#749)

* Scroll to survey form errors.

* Fix vantage codes bug that returned an array with a null element (#750)

* Sims taxonomy (#744)

* app is working with the taxonomy service

* updated snapshots

* tests

* survey service set up, using it for survey list

* skipping broken tests

* Remove `wldtaxonomic_units` table from survey queries

* merge conflicts fixed

* skipped some tests

* add TODO comment

* Trigger Build

* Fix some tests

* fixes attachmentList

* fixed tests

* Update project page test

* surveysList test updated

* Update ProjectsListPage.test.tsx

* Update ReviewAccessRequestForm.test.tsx

* test for Active Users List

* fix more tests

* code smells

* Remove unused imports

* fix general information bug

* fix warning on general information form

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Kjartan <[email protected]>

* BHBC-1689: Implement API Response Validation (#737)

* Add api response validation

* Updates

* update reponse for the project attachments list

* BHBC-1689: Add validation to projects

* BHBC-1689: Add response validation to surveys + attachments + observations

* BHBC-1689: Add response validation to project attachments.

* BHBC-1689: Fix publish_year type for report attachment uploading via POST.

* BHBC-1689: Fix api response validation for user data response.

* BHBC-1689: Change year_published type from string to number in attachment tests.

* BHBC-1689: Update tests

* Update occurrence submission response openapi

* BHBC-1689: Fix api response validation for uploads

* Update summary submission openapi and parse logic

* BHBC-1689: Fix all failing tests for reports

* BHBC-1689: Make lint-fix and make format-fix.

* BHBC-1689: Remove console logs used for testing.

* api validation for view.ts and surveys.ts

* BHBC-1689: Fix validation for surveys.

* BHBC-1689: Fix additional survey validation errors re publish_date.

* BHBC-1689: Fix additional survey validation for purpose_and_methodolgy and proprietary_data.

Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>

* Add prettier organize imports plugin (#747)

* Add prettier organize imports plugin

* Run formatter

* move permit endpoint functions to service (#751)

* move permit endpoint functions to service

* trigger openshift deploy

* fix code smell

* code smell v2

* Adjust elastic search query config to handle query terms with spaces (#752)

* Adjust elastic search query config to handle query terms with spaces

* Remove whole word search as it doesnt work, using this style anyways.

* BHBC-1720: Added required fields indicators (#756)

* BHBC-1720: Added asterisk to Project Boundary heading.

* BHBC-1720: Add required indicator for surveyed_all_areas (not finalized).

* BHBC-1720: Changed to 'Surveyed Areas *'

* BHBC-1720: Update snapshot tests

* Hide MU regions from the display list (#754)

* removed MU regions list from the display

* update api_delete_survey sql function (#757)

* date picker fix (#753)

* Bhbc-1725 fix deleting funding agency error (#758)

* fix funding_source delete error

* Bhbc 1728 (#760)

Misc UI Clean-up
Fixed the report year issue
Fixing version to string

* BHBC-1731: Fix open api spec (#762)

Fix open api specs for survey-view, survey funding sources update, and project-view

* Fix permit dropdown (#759)

* remove the ability to select a non-sampling permit (when editing a project)
* tweak openapi spec for public project view

* More openapi fixes (#764)

* public attachments received attachment/reports && secured/unsecured correctly
* fixed api calls for retrieve signedUrl and to get unsecured files

* Email to outlook (#765)

* fixed email redirect in header
* make agency_project_id nullable
* remove permit from the public page

* BHBC-1724: Fix edit dialog control clipping. (#761)

Fix edit dialog control clipping

* BHBC-1727: Add 'Zoom to Boundary Extent' button to SurveyStudyArea. (#763)

* BHBC-1727: Add 'Zoom to Boundary Extent' button to SurveyStudyArea.

* BHBC-1727: Automatically zoom to boundary extent upon rendering survey details.

* BHBC-1727: make lint-fix; make format-fix;

* BHBC-1727: Make Zoom to Boundary Extent button appear on top of the map

* BHBC-1727: Fix codesmell.

* BHBC-1727: Use absolute 'Zoom to boundary extent' IconButton in survey area map editor dialog.

* BHBC-1727: Add 'Zoom to boundary extent' IconButton to Project Location map.

* BHBC-1727: lint-fix and format-fit

* BHBC-1727: Made PR requested changes.

* BHBC-1727: Make lint-fix.

* BHBC-1727: Update snapshot tests

Co-authored-by: Anissa Agahchen <[email protected]>

* Fix user seed role id

* Move to test (#767)

* Update README.md

* First commit (#566)

* First commit

* Upload

* Update the sec produre for unseceruing an attachment

* Version update

* Update for single upload button

* New fixtures

* Update

* Survey script

* delete survey

* Spacing

* Update access request page: remove regional offices section, remove non-supported roles from dropdown (#610)

* [BHBC-1449] Add Help Link (#614)

* Update template validation schemas (#612)

* Add migration to update moose SRB/Composition Validation schema
* Add Summary Statistics file to resources page
* Add Date column to recruitment survey
* Fix dependency warnings
* Remove unused test function

* Update deployStatic.yml

* BHBC-1363 (#599)

Support project/survey report attachments with meta

* Add admin only endpoint to update the api log level at runtime (#619)

* Add admin only endpoint to update the api log level at runtime

* Update test

* Adding Test scripts (#617)

* Adding Test scripts

* Create project and Survey

* Remove second test file

* Test Updates/Fixes (#620)

Test updates/Fixes

* BHBC-1363-2: Fix signed url for reports, update fileupload to support single file only settings. (#621)

* BHBC-1461 (#622)

* Readme makefile updates (#623)

* Update Makefile, Readme, env
* Rename BioHub -> SIMS

* Add `Order By` clause to species sql (#627)

* BHBC-1462: Add new record to proprietary_type lookup table. (#629)

- Update code to leverage the `is_first_nation` column from the proprietary_type table.
  - Previously the code was using the row id

* BHBC-1470: View/edit survey and project metadata (#624)

BHBC-1470: View/edit survey and project metadata (#624)

* BHBC-1478: Update API to check project level permissions. (#630)

* Combine dependabot (#634)

* Bump tmpl from 1.0.4 to 1.0.5 in /app

Bumps [tmpl](https://github.com/daaku/nodejs-tmpl) from 1.0.4 to 1.0.5.
- [Release notes](https://github.com/daaku/nodejs-tmpl/releases)
- [Commits](https://github.com/daaku/nodejs-tmpl/commits/v1.0.5)

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

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

* Bump ws from 5.2.2 to 5.2.3 in /app

Bumps [ws](https://github.com/websockets/ws) from 5.2.2 to 5.2.3.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@5.2.2...5.2.3)

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

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

* Bump validator from 13.6.0 to 13.7.0 in /n8n

Bumps [validator](https://github.com/validatorjs/validator.js) from 13.6.0 to 13.7.0.
- [Release notes](https://github.com/validatorjs/validator.js/releases)
- [Changelog](https://github.com/validatorjs/validator.js/blob/master/CHANGELOG.md)
- [Commits](validatorjs/validator.js@13.6.0...13.7.0)

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

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

* Update

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* migration (#637)

* Bhbc 1470 (#638)

* Small test script and seeding changes (#631)

* Small test script and seeding changes

* Remove Shreyas from the seed

Co-authored-by: Nick Phura <[email protected]>

* Create zap.yml

* Update zap.yml

* Create dev.context

* Update zap.yml

* Zap follow up (#640)

* Small test script and seeding changes

* Remove Shreyas from the seed

* Details for ZAP

* Subtitute hardcoded PW with GHA- Secret

* [BHBC-1470] Increase Report Summary Input Size (#641)

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* Update zap.yml

* BHBC-1467: One page project/survey layout (#636)

One page project/survey layout

* Test Script update, login and some other details (#646)

* Test Script update, login and some other details

* Add delay to login

* Remove file

* Pointing to the right authurl

* Path confusion

* And again

* It is all in the timing

* Update readme as per Nick's comment

* BHBC-950 (#643)

- System admin users can delete other system users.

* Update DialogContext.Snackbar(#647)

Update snackbar to accept ReactNode as message

* Update deployStatic.yml

* [BHBC-950] UI Cleanup (#648)

* [BHBC-950] UI Cleanup

* [BHBC-950] Updating UI

* Trigger Build

* Update to test script to deal with changed id for report upload

* [BHBC-950] Fixing Request Access Table

Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: Anissa Agahchen <[email protected]>

* Update deployStatic.yml

* BHBC-949: Manage Project Team - Add + Delete User (#645)

- Add project users page
- Add project users "add project participant" form
- Add delete project user functionality
- Update some api project role authorization checks
- Update some frontend role check functionality
- misc fixes/updates

* Update deployStatic.yml

* First test (#650)

* First test

* Update path

* - update older node versions to 14
- Increase tsconfig versions
- Increase a few package.json versions
- Update a few git actions to latest versions

* Revert accidental backup file change

* Set back to es2015 otherwise the test framework will fail

Co-authored-by: Nick Phura <[email protected]>

* Update deployStatic.yml

* Manage system/project roles (#649)

* BHBC-948: Allow users to change existing system user roles.
* BHBC-1491: Allow users to change existing project participant's roles.

* Update deployStatic.yml

* [BHBC-948] UI Updates / Cleanup (#652)

* Updates to Agancies and Proeject Coordinator name/label

* Revert "Merge branch 'dev' of https://github.com/bcgov/biohubbc into dev"

This reverts commit 16b5f48, reversing
changes made to 1c88d34.

* Changes to codes.ts for removing BC and all over for the coordinator vs contact name … (#654)

* Cahnges to codes.ts and all over for the coordinator ro contact name change

* Update snapshots

* Finish BHBC-1505

* Update deployStatic.yml

* BHBC-1457 (#651)

Implemented EML endpoint and addition to target DwC archive file on S3. Moved all EML creation functionality to middle tier tooling.

* Tech debt (#653)

* Split up the workflow for uploading project/survey attachments and project/survey reports
* associated tests

* Updates to the test script and table changes (#658)

* All changes

* Update to formatting

* Remove new table

* Bhbc 1499 (#656)

Updates to the Moose template and transformation configs

* BHBC-1527: system and project roles updated (#659)

* update url for download (#660)

* Fix wonky survey view/view-for-update queries + misc (#664)

* Fix survey view/view-for-update queries + misc

* BHBC-1496: Transform Lat/Long to Darwin Core Accepted Format (#667)

- Enhance transformations to allow multiple source columns in parse step (uses first non-null value)
- Update scrape SQL to account for UTM or LatLong verbatimCoordinates
- Add `parseLatLongString` spatial-util function to parse a lat long string.

* BHBC-1525 (#669)

* BHBC-1525
- initial EML test file and low hanging target testing

* remove unused system accounts (#666)

* remove unused system accounts
* update code to reference previously updated role names

* Security rules for PorH (#668)

* Security rules for PorH

* Endpoint Tests (#661)

* Add missing `connection.commit`'s (#671)

* Update endpoint role authorization (#670)

* Update endpoint role authorization
* Update occurrence scrape verbatimCoordinates handling

* Bhbc 1400 (#657)

Goat template validations and transformations
Sheep template validations and transformations

* UI Updates / Clean-up (#674)

* Update all the BC and DC to tune the resource usage (#678)

* BHBC-1523: Add new users (#675)

* Added "+ new users" button on Admin user page and added testing for both API and app.

* Bhbc 1529- Add custom props to templates (#677)

* BHBC-1530: Update Moose transformations to account for all fields. (#680)

* BHBC-1530
- Enhance xlsx transformation parse step
- Update Moose validation/transformation schemas
- Minor refactor to part of transformation code
- Update resource page link

* BHBC-1520: View user details (#679)

- Ability to add system users from the manage users page
  - Associated endpoints and tests
- Page to view user details and their associated projects
  - Ability to remove user from projects or update their role.
  - Associated endpoints and tests
- Reduce some api code duplication

* BHBC-1531 (#683)

* BHBC-1531
    - adjusted taxonomic codes to exclude those codes that have end date setc
* Fix focal species yup schema validation rule

* BHBC-1520 (#684)

Prevent projects from having no Project Lead role: bug fixes + tests

* BHBC-1512: Added zoom for fullscreen map on project details. (#685)

Added zoom for fullscreen map on project details

* Enhance useInterval hook to support a timeout. Add unit tests. (#686)

* BHBC-1512-2: Additional map scroll wheel zoom changes (#687)

* BHBC-1512-2: Additional map scroll wheel zoom changes

* BHBC-1545: Tech Debt: Re-export all queries via queries.ts. Update custom error naming. (#688)

Add index files for all query folders
Add queries.ts to re-export all queries.
Update all references to existing queries to instead reference queries.ts
Rename `CustomError.ts->CustomError` to `custom-error.ts->HTTPError` and all references.
Run formatter/linter.

* BHBC-1545: Tech Debt: Updated apidoc for /user/* (#681)

Update openapi doc for /user/* endpoints
Misc unit test updates

* BHBC-1503: GCNotfiy (#689)

* Created GCNotify Api endpoint for generic email functionality

* BHBC-1557: Tech Debt 3 (#690)

* Add user-service.ts
Move user functions to user-service.ts.
Update tests.

Add code-service.ts
Move code functions to code-service.ts
Update tests.

Misc root-api-doc.ts cleanup

Add swagger-ui container to docker-compose

* Add SwaggerUI to docker-compose.yml (for local dev currently)

* Minor makefile updates

* Updates to error classes

* Updates to error logging

* Minor test tweak

* Handle DatabaseErrors

* remove console logs

* BHBC-1492: Email Access Request (#692)

* BHBC-1557: Tech Debt - Update App route handling and route security. (#693)

* Tech debt: Fix partnerships, and update /project/{projectId}/update endpoint (#694)

* BHBC-1557: Ke-tech-debt (#698)

* updated openApi schemas

* Added Survey creation. (#700)

* Unify dependabot (#701)

* Bump nanoid from 3.1.30 to 3.2.0 in /n8n (#704)

* Bump copy-props from 2.0.4 to 2.0.5 in /api (#702)

* Bump node-fetch from 2.6.5 to 2.6.7 in /n8n (#705)

* BHBC-1557: Add ProjectService. (#699)

* Bump lodash from 4.17.20 to 4.17.21 in /app (#703)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.20 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.20...4.17.21)

---
updated-dependencies:
- dependency-name: lodash
  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: Anissa Agahchen <[email protected]>

* BHBC-1493: Approval Email For Access Request  (#708)

Approval email sent to idir email connected to access request user.

* Dev ops  scale schemaspy up and down (#709)

* Scale SchemaSpy as part of the pipeline to make sure it is current all the time

* Make it a one step SchemaSpy cycle

* Formatting

* Fix typo

* Reduce cpu/request limits

* Revert "Reduce cpu/request limits"

This reverts commit b99f8cb.

* Reduce cpu/request limits + Adjust default Log Level (#716)

* BCEID bug fix + improvements (#717)

* BHBC-1613 - support bceid login (#720)

* BHBC-1580: Updates to moose/sheep/goat templates (#725)

* Tech debt 6 (#707)

* Disable html logging when running app tests.
* Update logger utils, make mocha disable winston logger before test execution
* Add missing type definition for @tmcw/togeojson
* Updates to map components (from resto)
* Update typescript and knex versions, remove swagger ui docker compose step, add swagger-ui directly to app
* Incorporate dependabot changes
* Update eslint config
* Run lint-fix using updated eslint rules and fix issues
* Update startend date component
* Move old `/users` endpoint to `/user/list`

* First cut of the postman collection with automatic login (#726)

* Isolating Service Code (#724)

Port changes from restoration to move functionality to services

* Update siteminder url (#736)

* Survey- Additional Fields (#731)

* create/view/edit  survey page with new purpose and methodology form
* addition of "ecological season", "intended outcome" and "vantage" lookup tables
& removal of common survey methodology - field table name change to "field method"
* "survey.objectives" attribute name change to "survey.additional_details"
*  seeding new lookup tables
* re-seed "field method" table
* tests

* package lock updates for minimist (#740)

* dependabot change version for moment (#739)

* Bump convict from 6.2.0 to 6.2.2 in /n8n (#745)

Bumps [convict](https://github.com/mozilla/node-convict) from 6.2.0 to 6.2.2.
- [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.0...v6.2.2)

---
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>

* Bump async from 2.6.3 to 2.6.4 in /app (#743)

Bumps [async](https://github.com/caolan/async) from 2.6.3 to 2.6.4.
- [Release notes](https://github.com/caolan/async/releases)
- [Changelog](https://github.com/caolan/async/blob/v2.6.4/CHANGELOG.md)
- [Commits](caolan/async@v2.6.3...v2.6.4)

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

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump async from 3.2.0 to 3.2.3 in /testing/e2e (#742)

Bumps [async](https://github.com/caolan/async) from 3.2.0 to 3.2.3.
- [Release notes](https://github.com/caolan/async/releases)
- [Changelog](https://github.com/caolan/async/blob/master/CHANGELOG.md)
- [Commits](caolan/async@v3.2.0...v3.2.3)

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

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* BHBC-1688: Add survey radio field (#746)

* BHBC-1688: Add new radio button field to survey.
* Add github action to write openshift urls

* Techdebt (#741)

remove default logs
fix tests
fix code smells

* Misc Cleanup (#748)

* misc cleanup
* Update user seed file

* BHBC-1690: Add scroll to error formik component (#749)

* Scroll to survey form errors.

* Fix vantage codes bug that returned an array with a null element (#750)

* Sims taxonomy (#744)

* app is working with the taxonomy service

* updated snapshots

* tests

* survey service set up, using it for survey list

* skipping broken tests

* Remove `wldtaxonomic_units` table from survey queries

* merge conflicts fixed

* skipped some tests

* add TODO comment

* Trigger Build

* Fix some tests

* fixes attachmentList

* fixed tests

* Update project page test

* surveysList test updated

* Update ProjectsListPage.test.tsx

* Update ReviewAccessRequestForm.test.tsx

* test for Active Users List

* fix more tests

* code smells

* Remove unused imports

* fix general information bug

* fix warning on general information form

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Kjartan <[email protected]>

* BHBC-1689: Implement API Response Validation (#737)

* Add api response validation

* Updates

* update reponse for the project attachments list

* BHBC-1689: Add validation to projects

* BHBC-1689: Add response validation to surveys + attachments + observations

* BHBC-1689: Add response validation to project attachments.

* BHBC-1689: Fix publish_year type for report attachment uploading via POST.

* BHBC-1689: Fix api response validation for user data response.

* BHBC-1689: Change year_published type from string to number in attachment tests.

* BHBC-1689: Update tests

* Update occurrence submission response openapi

* BHBC-1689: Fix api response validation for uploads

* Update summary submission openapi and parse logic

* BHBC-1689: Fix all failing tests for reports

* BHBC-1689: Make lint-fix and make format-fix.

* BHBC-1689: Remove console logs used for testing.

* api validation for view.ts and surveys.ts

* BHBC-1689: Fix validation for surveys.

* BHBC-1689: Fix additional survey validation errors re publish_date.

* BHBC-1689: Fix additional survey validation for purpose_and_methodolgy and proprietary_data.

Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>

* Add prettier organize imports plugin (#747)

* Add prettier organize imports plugin

* Run formatter

* move permit endpoint functions to service (#751)

* move permit endpoint functions to service

* trigger openshift deploy

* fix code smell

* code smell v2

* Adjust elastic search query config to handle query terms with spaces (#752)

* Adjust elastic search query config to handle query terms with spaces

* Remove whole word search as it doesnt work, using this style anyways.

* BHBC-1720: Added required fields indicators (#756)

* BHBC-1720: Added asterisk to Project Boundary heading.

* BHBC-1720: Add required indicator for surveyed_all_areas (not finalized).

* BHBC-1720: Changed to 'Surveyed Areas *'

* BHBC-1720: Update snapshot tests

* Hide MU regions from the display list (#754)

* removed MU regions list from the display

* update api_delete_survey sql function (#757)

* date picker fix (#753)

* Bhbc-1725 fix deleting funding agency error (#758)

* fix funding_source delete error

* Bhbc 1728 (#760)

Misc UI Clean-up
Fixed the report year issue
Fixing version to string

* BHBC-1731: Fix open api spec (#762)

Fix open api specs for survey-view, survey funding sources update, and project-view

* Fix permit dropdown (#759)

* remove the ability to select a non-sampling permit (when editing a project)
* tweak openapi spec for public project view

* More openapi fixes (#764)

* public attachments received attachment/reports && secured/unsecured correctly
* fixed api calls for retrieve signedUrl and to get unsecured files

* Email to outlook (#765)

* fixed email redirect in header
* make agency_project_id nullable
* remove permit from the public page

* BHBC-1724: Fix edit dialog control clipping. (#761)

Fix edit dialog control clipping

* BHBC-1727: Add 'Zoom to Boundary Extent' button to SurveyStudyArea. (#763)

* BHBC-1727: Add 'Zoom to Boundary Extent' button to SurveyStudyArea.

* BHBC-1727: Automatically zoom to boundary extent upon rendering survey details.

* BHBC-1727: make lint-fix; make format-fix;

* BHBC-1727: Make Zoom to Boundary Extent button appear on top of the map

* BHBC-1727: Fix codesmell.

* BHBC-1727: Use absolute 'Zoom to boundary extent' IconButton in survey area map editor dialog.

* BHBC-1727: Add 'Zoom to boundary extent' IconButton to Project Location map.

* BHBC-1727: lint-fix and format-fit

* BHBC-1727: Made PR requested changes.

* BHBC-1727: Make lint-fix.

* BHBC-1727: Update snapshot tests

Co-authored-by: Anissa Agahchen <[email protected]>

* Fix user seed role id

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Shrey Patel <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>

* BHBC-1700: Move to Test (#774)

* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

Co-authored-by: Curtis Upshall <[email protected]>

* BHBC-1700: Move to Test (#777)

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Shrey Patel <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>
NickPhura added a commit that referenced this pull request Sep 29, 2022
* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
NickPhura added a commit that referenced this pull request Nov 17, 2022
* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

* BHBC-1925: added permit servce test coverage (#824)

* BHBC-1904 (#822)

Removed all public views/ pages and supporting tests/ SQL. All unauthenticated users will now be directed to the login page

* BHBC-1949: Email notification to Users that access has been granted or denied

* BHBC-1950/BHBC-1951: Import/Validate Deer Survey Templates (#832)

* BHBC-1965: modifications to occurrence and summary details data model (#830)

 - addition of species descriptor to survey occurrence templates
- addition of templates concept to survey summary details

- adjusted file name

Co-authored-by: Kjartan <[email protected]>

* email link functioning (#833)

* BHBC-1937 - Enhance MapContainer with changes from BioHub (#827)

BHBC-1937: Ported MapContainer improvements over from platform:
 * Created new React components to encapsulate Map components and `wfs-utils` helper for rendering and selecting regional/mu layers
 * Updated tests and added type defs for map util functions.

* BHBC-1962-1963-1982-1975: Validation refactoring and error handling (#828)

* BHBC-1978: Data Administrator Permissions Are Incorrect (#838)

* BHBC-1972: Delete a Draft Project (#837)

* BHBC-1998, BHBC-1999 (#839)

Refactored Darwin core submission endpoint, added column to track Darwin core data as JSON

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC:1992: bugfix - keycloak env not passed to api (#840)

Fixed keycloak envs so they are passed properly to API

* updated file name (#841)

Renamed migration to maintain standards

* BHBC-1979: Update Templates to Use New Identifiers (#835)

* BHBC 2009 (#836)

Added wrapped function to catch any errors thrown by database functions

* BHBC-2018 (#843) Addition of security review timestamp attribute for all documents

* BHBC-2052 (#847)

Fixed keycloak URL for SIMS/ BioHub intake

* PR fix (#844)

* BHBC-1976: Update API to utilize new DB changes for processing summary result templates (#834)

 * BHBC-1976: Enhance summary template validation to get validation schema from database.
 * Created a service and repository file for summary template validation.
 * Determine validation schema by performing search on template name, version and taxonomic code.
 * Read in template name and version from XLSX custom properties.

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
NickPhura added a commit that referenced this pull request Nov 17, 2022
* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

* BHBC-1925: added permit servce test coverage (#824)

* BHBC-1904 (#822)

Removed all public views/ pages and supporting tests/ SQL. All unauthenticated users will now be directed to the login page

* BHBC-1949: Email notification to Users that access has been granted or denied

* BHBC-1950/BHBC-1951: Import/Validate Deer Survey Templates (#832)

* BHBC-1965: modifications to occurrence and summary details data model (#830)

 - addition of species descriptor to survey occurrence templates
- addition of templates concept to survey summary details

- adjusted file name

Co-authored-by: Kjartan <[email protected]>

* email link functioning (#833)

* BHBC-1937 - Enhance MapContainer with changes from BioHub (#827)

BHBC-1937: Ported MapContainer improvements over from platform:
 * Created new React components to encapsulate Map components and `wfs-utils` helper for rendering and selecting regional/mu layers
 * Updated tests and added type defs for map util functions.

* BHBC-1962-1963-1982-1975: Validation refactoring and error handling (#828)

* BHBC-1978: Data Administrator Permissions Are Incorrect (#838)

* BHBC-1972: Delete a Draft Project (#837)

* BHBC-1998, BHBC-1999 (#839)

Refactored Darwin core submission endpoint, added column to track Darwin core data as JSON

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC:1992: bugfix - keycloak env not passed to api (#840)

Fixed keycloak envs so they are passed properly to API

* updated file name (#841)

Renamed migration to maintain standards

* BHBC-1979: Update Templates to Use New Identifiers (#835)

* BHBC 2009 (#836)

Added wrapped function to catch any errors thrown by database functions

* BHBC-2018 (#843) Addition of security review timestamp attribute for all documents

* BHBC-2052 (#847)

Fixed keycloak URL for SIMS/ BioHub intake

* PR fix (#844)

* BHBC-1976: Update API to utilize new DB changes for processing summary result templates (#834)

 * BHBC-1976: Enhance summary template validation to get validation schema from database.
 * Created a service and repository file for summary template validation.
 * Determine validation schema by performing search on template name, version and taxonomic code.
 * Read in template name and version from XLSX custom properties.

* commit empty 2

* commit empty 3

* commit empty 4

* change readme

* change readme 2

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
KjartanE added a commit that referenced this pull request Nov 17, 2022
* BHBC-1966: Push Dev To Test (#826)

* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>

* Clean-up / Refactor Project Details and Related Components

* Update Project List Page / Remove Unused Functionality from Attachments List

* Standardize Title Container Structure and Styling

* Cleanup Title Container Alignment and Spacing

* Resource Page Clean-up

* Resource Page Clean-up

* Manage Users Page Clean-up

* Users Interface Clean-up

* dialog setup

* Formatting Updates

* Project Details and Component Clean-up

* Edit Page functioning errors still present

* UI Layout Updates

* UI Cleanup

* Clean-up Front-end for Edit Project Page

* UI Updates for Create / Edit Project UI's

* fix iucn and funding edits

* Project Detail Component Clean-up

* UI Clean-up

* Fix one line

* fix save and cancel buttons

* Spacing clean-up

* Test rework more work to be done

* Test to Dev (#850)

Prepping dev for test release

* User Details Page Cleanup

* Fix Naming Conventions

* Finish fix all broken tests

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
KjartanE added a commit that referenced this pull request Nov 17, 2022
* BHBC-1966: Push Dev To Test (#826)

* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>

* Clean-up / Refactor Project Details and Related Components

* Update Project List Page / Remove Unused Functionality from Attachments List

* Standardize Title Container Structure and Styling

* Cleanup Title Container Alignment and Spacing

* Resource Page Clean-up

* Resource Page Clean-up

* Manage Users Page Clean-up

* Users Interface Clean-up

* dialog setup

* Formatting Updates

* Project Details and Component Clean-up

* Edit Page functioning errors still present

* UI Layout Updates

* UI Cleanup

* Clean-up Front-end for Edit Project Page

* UI Updates for Create / Edit Project UI's

* fix iucn and funding edits

* BHBC-2052 (#847)

Fixed keycloak URL for SIMS/ BioHub intake

* PR fix (#844)

* Project Detail Component Clean-up

* UI Clean-up

* Fix one line

* fix save and cancel buttons

* Spacing clean-up

* BHBC-1976: Update API to utilize new DB changes for processing summary result templates (#834)

 * BHBC-1976: Enhance summary template validation to get validation schema from database.
 * Created a service and repository file for summary template validation.
 * Determine validation schema by performing search on template name, version and taxonomic code.
 * Read in template name and version from XLSX custom properties.

* Test rework more work to be done

* Test to Dev (#850)

Prepping dev for test release

* User Details Page Cleanup

* Fix Naming Conventions

* Finish fix all broken tests

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
AlfredRosenthal added a commit that referenced this pull request Nov 23, 2022
* BHBC-2058: Edit Project as a Single Page / General UI Clean-up (#856)

* BHBC-1966: Push Dev To Test (#826)

* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>

* Clean-up / Refactor Project Details and Related Components

* Update Project List Page / Remove Unused Functionality from Attachments List

* Standardize Title Container Structure and Styling

* Cleanup Title Container Alignment and Spacing

* Resource Page Clean-up

* Resource Page Clean-up

* Manage Users Page Clean-up

* Users Interface Clean-up

* dialog setup

* Formatting Updates

* Project Details and Component Clean-up

* Edit Page functioning errors still present

* UI Layout Updates

* UI Cleanup

* Clean-up Front-end for Edit Project Page

* UI Updates for Create / Edit Project UI's

* fix iucn and funding edits

* Project Detail Component Clean-up

* UI Clean-up

* Fix one line

* fix save and cancel buttons

* Spacing clean-up

* Test rework more work to be done

* Test to Dev (#850)

Prepping dev for test release

* User Details Page Cleanup

* Fix Naming Conventions

* Finish fix all broken tests

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>

* Revert "BHBC-2058: Edit Project as a Single Page / General UI Clean-up (#856)" (#857)

This reverts commit 5c1cd58.

* BHBC-1097: Use attachment-service for getProjectAttachments and getProjectReportAttachments

* BHBC-1097: implement project attachment get methods in service and repo files

* BHBC-1097: Update project attachments list endpoint to use service methods

* BHBC-1097: Sepearate project attachments and report attachments

* BHBC-1097: Attachment repo and service typedef refactor

* BHBC-1097: Show status for project attachment

* BHBC-1097: Report status implemented for projects

* BHBC-1097: Run make fix

* BHBC-1097: Remove commented code

Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
curtisupshall added a commit that referenced this pull request Nov 25, 2022
* BHBC-2058: Edit Project as a Single Page / General UI Clean-up (#856)

* BHBC-1966: Push Dev To Test (#826)

* Fix/Enhancement: Port improved user access request functionality from 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.

* BHBC-1730: Use centerOfMass instead of centroid for project area center (#770)

* BHBC-1730: Use centerOfMass instead of centroid for project area center point.

* BHBC-1736: Fix Read More/Read Less idempotence bug (#772)

* BHBC-1736: Tighten qualification for string truncating

* BHBC-1736: Fix null coalesce

* BHBC-1736: Change variable name to be more descriptive

* Bugfix: Incorrect identity source enum (#776)

- account for the fact that openapi enums are case sensitive

* Create toDraft.yml (#779)

* Create toDraft.yml

* remove dependency

* Bump sqlite3 from 5.0.2 to 5.0.8 in /n8n (#773)

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 (#781)

* Fixed horizontal autoscaler and consolidate deployment files

* Some config changes

* Forgot one

* fix

* Set utilization as portion of reques, not maximum

* BHBC-1778: Make resources page publicly viewable (#783)

* BHBC-1776: Update docker-compose and related env vars (#782)

* BHBC-1759: Fix cypress tests (#784)

- Fix cypress tests
- Update n8n version/config

* BHBC-1755: Add backbone api env var and platform-service. (#780)

* 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

* Node upgrade as Cypress needs a newer version of node (#785)

* Bump convict from 6.2.2 to 6.2.3 in /n8n (#786)

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>

* Updates the HPAs for APP and API (#790)

* BHBC-1783: Error When Editing Purpose and Methodology (#787)

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.

* BHBC-1775, BHBC-1809: Templates in prod are still downloaded from S3 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-1810 : bug fix - xlsx observation upload not completing (#792)

BHBC-1796: Bug fix - Observation points not displaying on map
BHBC-1810: Bug fix - Transformation of large template file crashes API in Openshift

* fix display of occurrences on full-screen view (#794)

* BHBC-1789: EML Updates + Survey Service (#766)

Tech debt: survey-service
EML: major updates to EML creation code

* BUGFIX: Strip null values from vantage code array in SQL response

* Tech Debt: Fix Cypress Tests (#800)

Tech Debt: Fix Cypress Tests (#800)

* BHBC-1837: Fix survey attachment metadata endpoint (#801)

Bug fix for survey attachment view details

* BHBC-1201: Push project/survey metadata (EML) to backbone on project/survey create/update (#803)

BHBC-1201: Push project/survey metadata (EML) to backbone on create/update

* BHBC-1841: Remove Project/Survey Publish Buttons And Related Code (#804)

 - Project and survey UI no longer include publish button
 - API no longer has endpoints for publishing

* BHBC-1868: Push Observation Data to BioHub (#805)

System admin button + endpoint to submit project and survey EML + Observation data to BioHub Backbone.

* BHBC-1868-2: Handle error popup for BioHub Submit, remove some old logs, enable biohub submission env var for dev. (#806)

- Handle error popup for BioHub Submit
- remove some old logs
- enable biohub submission env var for dev

* TechDebt: Update Keycloak config - add pkce method (#807)

- Update keycloak config to include pkce option

* TechDebt: Port latest version of DataLoader hook (and related hooks/tests) to SIMS (#809)

* TechDebt: Update Template apiVersion from "v1" to "template.openshift.io/v1" (#810)

* BHBC-1893/1880: Update the Composition and SRB Survey Template (#808)

* BHBC-1867: Cant delete a system user that has no system role (#811)

- Fix issue that prevented deleting system users that had no system role.
- Fix bug that was displaying non-human users in the manage users list.

* Update Sims Template Validations (#812)

* BHBC-1914, 1918, 1920 (#816)

Updated validation for summary templates

* BHBC-1923: Update Resource Page (#814)

* BHBC-1912:  Remove the Table Preview for Submitted Observations and Summary Results (#815)

* edits for moose srb and comp validation

* add transect validation

* add sheep pop comp validation survey (broken)

* sheep validation functioning

* goat validation functioning

* fix edits for validations

* trigger build

* initial commit

* remove Observation and Summary CSV table and ref

* trigger build

* fix merge conflict

* more merge conflicts

Co-authored-by: Anissa Agahchen <[email protected]>

* BHBC-1922: Add Attachments Datra To EML (#817)

* Include attachments metadata in EML
- adjusted EML pub date to short ISO format
- adjusted order of EML to conform with specification
- adjusted database smoke test script for usage as initial data population. while application now does not throw error, issues persist and have been documented in Jira

* added new elk summary template to resource page (#818)

Added Elk Summary Template to Resource Page

* BHBC-1930, 1931, 1932 (#820)- Elk Templates and summary results

* BHBC-1547: Create Project Improvements (#819)

Refactor create project page to a single form, removing the multi-step wizard

* BHBC-1925: Move Permits to Survey (#813)

- Adjusted permit model for model 1.2
- Adjusted db function project deletion routines
- Added smoke test file
- Purge old permit code from all project related components (frontend and backend)
- Add CustomFieldArray Component
- Move permit components/logic to Survey (from Project)

* BHBC-1966: Add 2 new items to the agency codes list (#825)

Add 2 new items to the agency codes list

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: Kjartan <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>

* Clean-up / Refactor Project Details and Related Components

* Update Project List Page / Remove Unused Functionality from Attachments List

* Standardize Title Container Structure and Styling

* Cleanup Title Container Alignment and Spacing

* Resource Page Clean-up

* Resource Page Clean-up

* Manage Users Page Clean-up

* Users Interface Clean-up

* dialog setup

* Formatting Updates

* Project Details and Component Clean-up

* Edit Page functioning errors still present

* UI Layout Updates

* UI Cleanup

* Clean-up Front-end for Edit Project Page

* UI Updates for Create / Edit Project UI's

* fix iucn and funding edits

* Project Detail Component Clean-up

* UI Clean-up

* Fix one line

* fix save and cancel buttons

* Spacing clean-up

* Test rework more work to be done

* Test to Dev (#850)

Prepping dev for test release

* User Details Page Cleanup

* Fix Naming Conventions

* Finish fix all broken tests

Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>

* Revert "BHBC-2058: Edit Project as a Single Page / General UI Clean-up (#856)" (#857)

This reverts commit 5c1cd58.

* BHBC-1097: Use attachment-service for getProjectAttachments and getProjectReportAttachments

* BHBC-1097: implement project attachment get methods in service and repo files

* BHBC-1097: Update project attachments list endpoint to use service methods

* BHBC-1097: Sepearate project attachments and report attachments

* BHBC-1097: Attachment repo and service typedef refactor

* BHBC-1097: Show status for project attachment

* BHBC-1097: Report status implemented for projects

* BHBC-1097: Run make fix

* BHBC-1097: Remove commented code

* BHBC-1097: Separate chip into its own component file

* - rectified missing artifact ids in security association unique keys

Co-authored-by: Kjartan <[email protected]>
Co-authored-by: Nick Phura <[email protected]>
Co-authored-by: Roland Stens <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Anissa Agahchen <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: Charlie Garrett-Jones <[email protected]>
Co-authored-by: jeznorth <[email protected]>
Co-authored-by: AlfredRosenthal <[email protected]>
Co-authored-by: cgarrettjones <[email protected]>
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.

3 participants