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

Psp 7614, 7657, 7626, 7610 assorted text corrections. #3737

Merged
merged 5 commits into from
Jan 29, 2024

Conversation

devinleighsmith
Copy link
Collaborator

No description provided.

@devinleighsmith devinleighsmith added the bug Something isn't working label Jan 23, 2024
@devinleighsmith devinleighsmith self-assigned this Jan 23, 2024
Copy link
Contributor

✅ No secrets were detected in the code.

Copy link

codecov bot commented Jan 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8756e7d) 75.56% compared to head (a9a64fa) 75.56%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #3737   +/-   ##
=======================================
  Coverage   75.56%   75.56%           
=======================================
  Files        1447     1447           
  Lines       37533    37533           
  Branches     7058     7058           
=======================================
  Hits        28363    28363           
  Misses       8891     8891           
  Partials      279      279           
Flag Coverage Δ
unittests 75.56% <100.00%> (ø)

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

Files Coverage Δ
...rce/backend/api/Services/DispositionFileService.cs 77.77% <100.00%> (ø)
...ideBar/disposition/form/DispositionTeamSubForm.tsx 96.00% <ø> (ø)
.../tabs/offersAndSale/OffersAndSaleContainerView.tsx 84.09% <100.00%> (ø)
...ispositionOfferDetails/DispositionOfferDetails.tsx 91.66% <ø> (ø)

Copy link
Contributor

✅ No secrets were detected in the code.

Copy link
Contributor

✅ No secrets were detected in the code.

1 similar comment
Copy link
Contributor

✅ No secrets were detected in the code.

@devinleighsmith devinleighsmith merged commit 3374099 into bcgov:dev Jan 29, 2024
9 checks passed
FuriousLlama added a commit that referenced this pull request Jan 31, 2024
* Refactoring of modals and changes on Smoke Test Automation Test Cases

* Automation refactoring - modals, updating fields on Compensation, Digital Docs for Property Management

* Refactoring modals

* Automation refactoring - modals, updating fields on Compensation, Digital Docs for Property Management

* Refactoring modals

* Automation updates

* Automation updates

* Contacts elements update

* Removal of old frontend api files

* Improved checklist Api and backend services

* Moved association models to the api project and added code types to be shared with the frontend

* minor - backend import updates

* major - fixed backend file models inheritance missalingments

* major - fixed lease missing fields and improper inheritance

* major - fixed backend typos

* major - utilty method changes

* minor - updated type references. This commit can be skimed as it should not contain logic changes

* major - type updates and nullable conversions on form models

* major - type changes with possible logic implications. isValidId changes.

* minor - type code alignments and cleanup

* major - more type changes. Removal of adjacent land form model

* minor - test and mock updates

* generation code changesa

* manual merge

* minor - added nullable property to controllers that use nullable models

* major - lease fixes and removal of adjacent land code

* major - improved handling and validation of ID related values and dates. Now 0 and EpochIsoDate are considered invalid id and invalid date respectively

* major - test updates and regenerated snaps

* major - lease fixes around address

* minor - workspace change to force vscode to use the ts compiler specified in the package.json file

* major - fixed backend references and cyclical parsing due to the file model inheritance

* major - fixed disposition

* PSP-7509 : Allow Contractor to Create or Edit Disposition File (#3735)



Co-authored-by: Eduardo Herrera <[email protected]>

* CI: Bump version to v5.0.0-72.14

* Disposition File details, documents and notes. Refactoring on File Properties

* PSP-7510 : Disallow Contractor to Remove Themself from Disposition File (#3745)

Co-authored-by: Eduardo Herrera <[email protected]>

* CI: Bump version to v5.0.0-72.15

* correct disposition list view file number. (#3736)

* CI: Bump version to v5.0.0-72.16

* Offers and sales tab automation

* disposition offers and sales tab

* Psp 7614, 7657, 7626, 7610 assorted text corrections. (#3737)

* text corrections.

* correct title text.

* psp-7657 fix modal buttons.

* test updates.

* lint fixes.

* CI: Bump version to v5.0.0-72.17

* CI: Bump version to v5.0.0-72.18

* Refactoring and Disposition file automation

* Merge conflicts between local and remote

* Disposition files - Offers and sale create and update

* Deleting AcquisitionProperties class, replaced by SharedFileProperties

* Text changes (#3752)

* CI: Bump version to v5.0.0-72.19

* correct message text. (#3749)

* CI: Bump version to v5.0.0-72.20

* Refactoring LeaseTenants class

* CI: Bump version to v5.0.0-72.21

* Manual merge fixes

* Addressed pr comments

* fixed date formatting

---------

Co-authored-by: Sue Tairaku <[email protected]>
Co-authored-by: Sue Tairaku <[email protected]>
Co-authored-by: Alejandro Sanchez <[email protected]>
Co-authored-by: Eduardo <[email protected]>
Co-authored-by: Eduardo Herrera <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: devinleighsmith <[email protected]>
Co-authored-by: praveen <[email protected]>
Co-authored-by: pravkuma14 <[email protected]>
devinleighsmith added a commit that referenced this pull request Feb 1, 2024
* Refactoring of modals and changes on Smoke Test Automation Test Cases

* Automation refactoring - modals, updating fields on Compensation, Digital Docs for Property Management

* Refactoring modals

* Automation refactoring - modals, updating fields on Compensation, Digital Docs for Property Management

* Refactoring modals

* Automation updates

* Automation updates

* Contacts elements update

* Digital Documents subtitles fix

* CI: Bump version to v5.0.0-71.25

* Version bump (#3729)

* CI: Bump version to v5.0.0-72.1

* Disposition Files initial Automation

* CI: Bump version to v5.0.0-72.2

* Psp 7611 - ensure undefined/null handled correctly by contact parsers. (#3730)

* handle null and undefined contact fields.

* remove debugger;

* CI: Bump version to v5.0.0-72.3

* PSP-7578 Remove Project and Product from Header of Disposition File (#3734)

* Fix runtime error when user guid is invalid

* Remove Project and Product from Header of Disposition File

* Test corrections

* CI: Bump version to v5.0.0-72.4

* psp-7630 ensure that properties added directly to inventory have the … (#3728)

* psp-7630 ensure that properties added directly to inventory have the correct flags set.

* test correction.

Signed-off-by: devinleighsmith <[email protected]>

---------

Signed-off-by: devinleighsmith <[email protected]>

* CI: Bump version to v5.0.0-72.5

* psp-7609 ensure that changing the status of an acq file generates a note. (#3726)

* CI: Bump version to v5.0.0-72.6

* lint correction. (#3733)

* CI: Bump version to v5.0.0-72.7

* IS-72.00 Database Schema (#3716)

* IS-72.00 Database Schema

PSP_PIMS S72.00 Sprint 72 2024-Jan-16
- Altered tables:
  - PIMS_PROPERTY
  - PIMS_PROPERTY_HIST
- Altered build scripts:
  - 002_PSP_PIMS_Build.sql
  - 020_DML_PIMS_STATIC_VARIABLE.sql
  - 148_DML_PIMS_DSP_CHKLST_ITEM_TYPE.sql
- Added test scripts:
  - 071_PIMS_PROPERTY_CONTACT.sql
- Altered test scripts:
  - 005_DML_LEASE_PROPERTY.sql
- Requires additional metadata to meet standards

* fixed build not matching alter up result

* aligned latest folder

---------

Co-authored-by: Manuel Rodriguez <[email protected]>

* CI: Bump version to v5.0.0-72.8

* generated scaffold and removed lingering files (#3738)

* CI: Bump version to v5.0.0-72.9

* Disposition checklist automation

* fixed the disposition bug 7582

* CI: Bump version to v5.0.0-72.10

* Added Reordering of the DISPLAY_ORDER Columns (#3742)

Reordered the display order following the insertion/deletion of the PIMS_DSP_CHKLST_ITEM_TYPE table.

* CI: Bump version to v5.0.0-72.11

* PSP-7296 : Create/Edit Sales Details (Disposition) (#3731)

* CI: Bump version to v5.0.0-72.12

* PSP-7639 PIMS HIST Table fix commit (#3743)

Fix on missing Historical rows that were identified on 3 tables in UAT database

1.PIMS_INSURANCE

2.PIMS_LEASE_TENANT

3.PIMS_PROPERTY_LEASE

* CI: Bump version to v5.0.0-72.13

* PSP-7509 : Allow Contractor to Create or Edit Disposition File (#3735)



Co-authored-by: Eduardo Herrera <[email protected]>

* CI: Bump version to v5.0.0-72.14

* Disposition File details, documents and notes. Refactoring on File Properties

* PSP-7510 : Disallow Contractor to Remove Themself from Disposition File (#3745)

Co-authored-by: Eduardo Herrera <[email protected]>

* CI: Bump version to v5.0.0-72.15

* correct disposition list view file number. (#3736)

* CI: Bump version to v5.0.0-72.16

* Offers and sales tab automation

* disposition offers and sales tab

* Psp 7614, 7657, 7626, 7610 assorted text corrections. (#3737)

* text corrections.

* correct title text.

* psp-7657 fix modal buttons.

* test updates.

* lint fixes.

* CI: Bump version to v5.0.0-72.17

* CI: Bump version to v5.0.0-72.18

* Refactoring and Disposition file automation

* Merge conflicts between local and remote

* Disposition files - Offers and sale create and update

* Deleting AcquisitionProperties class, replaced by SharedFileProperties

* Text changes (#3752)

* CI: Bump version to v5.0.0-72.19

* correct message text. (#3749)

* CI: Bump version to v5.0.0-72.20

* Refactoring LeaseTenants class

* CI: Bump version to v5.0.0-72.21

* PSP-6668 FT: Acquisition File: Compensation: Show error modal instead of toast when total allowable compensation value is less than total compensation value (#3755)

* Show modal popup with warning message for total allowable compensation

* Test corrections

* CI: Bump version to v5.0.0-72.22

* PSP-7664 FT-REG: When cancelling, the confirmation modal should be consistent around different types of files (#3747)

* Increase the default width of modals

* Rename file

* Refactor duplicate code

* Verbiage update

* Update snapshots

* Text updates

* Test corrections

* CI: Bump version to v5.0.0-72.23

* psp-7680 add agreement status type to agreement export. (#3751)

* add agreement status type to agreement export.

* test corrections.

* CI: Bump version to v5.0.0-72.24

* Pipeline updates (#3756)

* add keycloak sync to all pipelines.
add uat hotfix and pre-release hotfix pipelines.
update builds/keycloak sync to support new configuration.

* correct uat secret name

* add prod secrets for keycloak sync

* correct typo.

* CI: Bump version to v5.0.0-72.25

---------

Signed-off-by: devinleighsmith <[email protected]>
Co-authored-by: Sue Tairaku <[email protected]>
Co-authored-by: Sue Tairaku <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: praveen <[email protected]>
Co-authored-by: devinleighsmith <[email protected]>
Co-authored-by: Doug Filteau <[email protected]>
Co-authored-by: Manuel Rodriguez <[email protected]>
Co-authored-by: pravkuma14 <[email protected]>
Co-authored-by: Eduardo <[email protected]>
Co-authored-by: ap-quartech <[email protected]>
Co-authored-by: Eduardo Herrera <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants