PSP-6824 Property management view

[asanchezr]

[github-actions]

✅ No secrets were detected in the code.

[codecov]

Codecov Report

Merging #3504 (430887a) into dev (6817697) will increase coverage by 4.05%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##              dev    #3504      +/-   ##
==========================================
+ Coverage   71.24%   75.30%   +4.05%     
==========================================
  Files        1340      877     -463     
  Lines       31955    17774   -14181     
  Branches     6005     4993    -1012     
==========================================
- Hits        22767    13384    -9383     
+ Misses       8941     4390    -4551     
+ Partials      247        0     -247     

Flag	Coverage Δ
unittests	75.30% <86.36%> (+4.05%)	⬆️

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

Files	Coverage Δ
...tend/src/components/common/MultiselectTextList.tsx	100.00% <100.00%> (ø)
source/frontend/src/components/common/TabView.tsx	100.00% <ø> (ø)
...ilsManagement/detail/PropertyManagementTabView.tsx	66.66% <ø> (ø)
...tail/summary/PropertyManagementDetailContainer.tsx	90.00% <100.00%> (+5.38%)	⬆️
...nt/detail/summary/PropertyManagementDetailView.tsx	100.00% <100.00%> (+28.57%)	⬆️
source/frontend/src/utils/formUtils.ts	81.94% <100.00%> (ø)
source/frontend/src/utils/test-utils.tsx	84.84% <50.00%> (-0.72%)	⬇️
source/frontend/src/utils/propertyUtils.ts	91.48% <75.00%> (-3.39%)	⬇️

... and 464 files with indirect coverage changes

[github-actions]

✅ No secrets were detected in the code.

[github-actions]

✅ No secrets were detected in the code.

[devinleighsmith]

This is minor, but the spacing seems a bit off the mockup. Can we move the field values a bit more to the left or was this done to be consistent with another screen?

[github-actions]

✅ No secrets were detected in the code.

[devinleighsmith]

On my screen I'm getting two scroll bars, are you seeing that @asanchezr

[asanchezr]

On my screen I'm getting two scroll bars, are you seeing that @asanchezr

I did not see this but good catch. This must be a bug within the TAB container not this story but will fix here

[asanchezr]

This is minor, but the spacing seems a bit off the mockup. Can we move the field values a bit more to the left or was this done to be consistent with another screen?

I have used the default spacing that comes with the <SectionField /> component - like other forms/views in the app. I would lean towards leaving as is for consistency within the app. Most likely UXpin spacing is not consistent with other forms. Like for example, the property details view uses the same spacing

[github-actions]

✅ No secrets were detected in the code.

[github-actions]

✅ No secrets were detected in the code.