Skip to content

Commit

Permalink
Fix validations for Misc Transfers (#1861)
Browse files Browse the repository at this point in the history
* Fix validations for Misc Transfers
  • Loading branch information
dimak1 authored May 8, 2024
1 parent 7e25bb7 commit 84ebade
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions ppr-ui/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion ppr-ui/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "ppr-ui",
"version": "3.1.15",
"version": "3.1.16",
"private": true,
"appName": "Assets UI",
"sbcName": "SBC Common Components",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1086,7 +1086,7 @@ export default defineComponent({
return TransSaleOrGift.hasMixedOwnersInGroup(groupId) || TransSaleOrGift.hasPartlyRemovedEATOwners(groupId)
}
if (isTransferWithoutBillOfSale.value) {
if (isTransferWithoutBillOfSale.value && !TransToExec.hasAllCurrentOwnersRemoved(groupId)) {
// check only for mixed owners, removed Exec, Admin, Trustees should not trigger the error
return TransSaleOrGift.hasMixedOwnersInGroup(groupId)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
</span>
</span>
<!-- Transfer to Admin error messages -->
<span v-else-if="isTransferWithoutBillOfSale">
<span v-else-if="isTransferWithoutBillOfSale && !TransToExec.hasAllCurrentOwnersRemoved(groupId)">
{{ hasOneHomeOwnerGroup ?
MixedRolesErrors.hasMixedOwnerTypes :
MixedRolesErrors.hasMixedOwnerTypesInGroup }}
Expand Down

0 comments on commit 84ebade

Please sign in to comment.