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

5.7 release to UAT #4565

Merged
merged 76 commits into from
Jan 2, 2025
Merged

5.7 release to UAT #4565

merged 76 commits into from
Jan 2, 2025

Conversation

devinleighsmith
Copy link
Collaborator

No description provided.

stairaku and others added 30 commits December 18, 2024 10:41
* Bump version IS-96

* Lint fixes
PSP_PIMS | Development Sprint: S96.00 | Design Sprint: 95 | Date: 2024-Dec-17
- Added tables:
  - PIMS_ACQ_FILE_ACQ_FL_TAKE_TYP
  - PIMS_ACQ_FILE_ACQ_FL_TAKE_TYP_HIST
  - PIMS_ACQ_FILE_ACQ_PROGRESS
  - PIMS_ACQ_FILE_ACQ_PROGRESS_HIST
  - PIMS_ACQ_FILE_APPRAISAL_TYPE
  - PIMS_ACQ_FILE_EXPROP_RISK_TYPE
  - PIMS_ACQ_FILE_LGL_SRVY_TYPE
  - PIMS_ACQ_FILE_PROGESS_TYPE
  - PIMS_ACQ_FILE_TAKE_TYPE
- Altered tables:
  - PIMS_ACQUISITION_FILE
  - PIMS_ACQUISITION_FILE_HIST
  - PIMS_ACQ_CHKLST_SECTION_TYPE
  - PIMS_DOCUMENT
  - PIMS_DOCUMENT_QUEUE
  - PIMS_DOCUMENT_QUEUE_HIST
  - PIMS_PROJECT_PERSON
  - PIMS_PROJECT_PERSON_HIST
  - PIMS_PROPERTY
- Added seed scripts:
  - 159_DML_PIMS_ACQ_FILE_PROGESS_TYPE.sql
  - 160_DML_PIMS_ACQ_FILE_APPRAISAL_TYPE.sql
  - 161_DML_PIMS_ACQ_FILE_LGL_SRVY_TYPE.sql
  - 162_DML_PIMS_ACQ_FILE_TAKE_TYPE.sql
  - 163_DML_PIMS_ACQ_FILE_EXPROP_RISK_TYPE.sql
- Requires additional metadata to meet standards

Co-authored-by: Manuel Rodriguez <[email protected]>
PSP-9710, PSP-9313: UI/UX Cleanup - Make icons consistent across PIMS
PSP-9211 and PSP-9719: UI/UX Clean up: Map Icons Replacement and PIMS File header colors
devinleighsmith and others added 17 commits December 30, 2024 13:34
* Address race condition where:
1) (thread #1) upload document action starts upload, sets document to processing
2) (thread #2) document poll starts executing with document in processing state
3) (thread #1) upload document action completes upload, sets document to sucess.
4) (thread #2) document poll throws concurrency exception.

* correction.

* test updates.
Co-authored-by: Herrera <[email protected]>
Co-authored-by: devinleighsmith <[email protected]>
PSP-9471: UI UX Clean Up - Contact – Manage Contacts: Allow to save contacts that are set as Inactive without Contact Methods
* fix infinite render loop caused by hasClaim function.

* formatting fixes.
* reduce round trips to database, add capability for database to retry transitive failures.

* Correct job name in appsettings - to ensure correct scheduled interval.

* test corrections.
… details after user adds the Project Team member (#4563)

Co-authored-by: Herrera <[email protected]>
Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

1 similar comment
Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

asanchezr and others added 2 commits January 2, 2025 02:17
…toral District (#4564)

* Code cleanup

* PSP-9683 Return compensation property locations in lat/long

* Test updates
Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

1 similar comment
Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

Copy link

codecov bot commented Jan 2, 2025

Codecov Report

Attention: Patch coverage is 67.91569% with 137 lines in your changes missing coverage. Please review.

Project coverage is 86.74%. Comparing base (6362ad3) to head (b0e44af).
Report is 77 commits behind head on test.

Files with missing lines Patch % Lines
...end/api/Areas/Documents/DocumentQueueController.cs 0.00% 35 Missing ⚠️
...rce/backend/dal/Repositories/DocumentRepository.cs 29.16% 17 Missing ⚠️
...ackend/dal/Repositories/DocumentQueueRepository.cs 77.14% 13 Missing and 3 partials ⚠️
...ource/backend/dal/Repositories/LookupRepository.cs 0.00% 15 Missing ⚠️
...kend/dal/Repositories/AcquisitionFileRepository.cs 58.33% 8 Missing and 2 partials ⚠️
source/backend/api/Services/DocumentFileService.cs 74.28% 6 Missing and 3 partials ⚠️
source/backend/api/Services/DocumentService.cs 0.00% 8 Missing ⚠️
...pi/Helpers/Extensions/AcquisitionFileExtensions.cs 50.00% 3 Missing and 3 partials ⚠️
....api/Repositories/RestCommon/BaseRestRepository.cs 0.00% 3 Missing ⚠️
.../Repositories/AcquisitionFileDocumentRepository.cs 0.00% 3 Missing ⚠️
... and 11 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             test    #4565      +/-   ##
==========================================
+ Coverage   86.71%   86.74%   +0.03%     
==========================================
  Files        1659     1669      +10     
  Lines      141147   142918    +1771     
  Branches    10582    10679      +97     
==========================================
+ Hits       122390   123976    +1586     
- Misses      18416    18585     +169     
- Partials      341      357      +16     
Flag Coverage Δ
unittests 86.74% <67.91%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...n/Controllers/CompensationRequisitionController.cs 0.00% <ø> (ø)
.../Areas/Documents/DocumentRelationshipController.cs 0.00% <ø> (ø)
source/backend/api/Controllers/LookupController.cs 100.00% <100.00%> (ø)
...rce/backend/api/Services/AcquisitionFileService.cs 74.63% <100.00%> (+0.08%) ⬆️
...end/api/Services/CompensationRequisitionService.cs 71.08% <100.00%> (+3.32%) ⬆️
...ource/backend/api/Services/DocumentQueueService.cs 100.00% <100.00%> (+100.00%) ⬆️
source/backend/api/Services/FormDocumentService.cs 100.00% <100.00%> (ø)
source/backend/api/Services/ResearchFileService.cs 79.08% <100.00%> (ø)
...ource/backend/apimodels/Models/Base/CodeTypeMap.cs 100.00% <ø> (ø)
...els/Concepts/AcquisitionFile/AcquisitionFileMap.cs 100.00% <100.00%> (ø)
... and 35 more

... and 64 files with indirect coverage changes

Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

1 similar comment
Copy link
Contributor

github-actions bot commented Jan 2, 2025

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4565

Copy link
Collaborator

@JamesPayer JamesPayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UAT release 5.7

@JamesPayer JamesPayer merged commit c76126a into test Jan 2, 2025
47 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants