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-7039 Take SRW Expiry Date #3565

Merged
merged 7 commits into from
Nov 2, 2023

Conversation

asanchezr
Copy link
Collaborator

NOTE: Unit tests will be added in an upcoming PR

Take Details

take-detail

Take Update form

take-update

@asanchezr asanchezr added the enhancement New feature or request label Nov 2, 2023
@asanchezr asanchezr self-assigned this Nov 2, 2023
Copy link
Contributor

github-actions bot commented Nov 2, 2023

✅ No secrets were detected in the code.

@@ -51,6 +52,8 @@ public void Register(TypeAdapterConfig config)
.Map(dest => dest.TakeTypeCode, src => src.TakeTypeCode)
.Map(dest => dest.TakeStatusTypeCode, src => src.TakeStatusTypeCode)
.Map(dest => dest.LandActTypeCode, src => src.LandActTypeCode.Id)
.Map(dest => dest.SrwEndDt, src => src.SrwEndDt)
.Map(dest => dest.IsAcquiredForInventory, () => true) // TODO: fix this
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@FuriousLlama - you will have to fix this mapping in your PR

Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Merging #3565 (94e2a14) into dev (16400dc) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #3565   +/-   ##
=======================================
  Coverage   69.50%   69.51%           
=======================================
  Files        1369     1369           
  Lines       33626    33634    +8     
  Branches     6250     6252    +2     
=======================================
+ Hits        23371    23379    +8     
  Misses      10004    10004           
  Partials      251      251           
Flag Coverage Δ
unittests 69.51% <100.00%> (+<0.01%) ⬆️

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

Files Coverage Δ
source/backend/api/Models/Concepts/Take/TakeMap.cs 100.00% <100.00%> (ø)
...rce/backend/api/Services/AcquisitionFileService.cs 73.54% <100.00%> (+0.09%) ⬆️
...Bar/property/tabs/takes/detail/TakesDetailView.tsx 92.85% <100.00%> (+0.17%) ⬆️
...SideBar/property/tabs/takes/update/TakeSubForm.tsx 77.92% <100.00%> (+0.29%) ⬆️
...Bar/property/tabs/takes/update/TakesUpdateForm.tsx 100.00% <ø> (ø)
...es/mapSideBar/property/tabs/takes/update/models.ts 90.19% <100.00%> (+0.19%) ⬆️

Copy link
Contributor

github-actions bot commented Nov 2, 2023

✅ No secrets were detected in the code.

@asanchezr asanchezr merged commit 4ca6964 into bcgov:dev Nov 2, 2023
9 checks passed
@asanchezr asanchezr deleted the psp-7039-take-expiry-date branch November 2, 2023 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants