UI - Bug - Staff Transfer Due to Sale/Gift - Issue when changing from SO to Tenants in Common

[tlebedovich]

This bug is most likely related to bug #18197

Issue when changing from SO to Tenants in Common (in TEST/DEV):

Bug : Previous Owner Group

See Quicktime video to see what I did to create the bug:

Private Zenhub Video

Steps I took:
STARTED a Transfer due to sale/gift with a sole owner
THEN added new owner to a group (group 2)
AND the original owner got automatically got moved into group 1 and interest set as N/A (as it should)
THEN I added new owner to Group 1 and in the process updated the N/A interest to 1/2 while adding this owner
NOW group 1 now contains the original existing owner plus a newly added owner.
WHEN I delete the original owner, they are automatically moved to their own "previous owners group" (as should happen)
THEN I deleted the new owner that was left in Group 1,
AND then deleted that empty Group 1
BUT instead of the empty Group 1 being removed completely from the table (since it no longer contained any existing owners) when I deleted the empty Group 1, it showed also as a "previous owner group" with a broken undo button (the bug)

Note: It feels like once a new owner is added to a group with an existing owner, even when that existing owner gets removed from that group, the group still acts like its an existing group instead of a new group.

Private Zenhub Image

[tlebedovich]

@mstanton1 - another staff sale/gift bug to add

[tlebedovich]

Found some further complications so will review this ticket again with fresh eyes on Monday. Will ping when its ready.

[tlebedovich]

ok @mstanton1 - this staff sale/gift transfer bug ticket is now ready to go

[mstanton1]

Hey team! Please add your planning poker estimate with Zenhub @cameron-eyds @chdivyareddy @dimak1 @doug-lovett @RuoxuanPengBC

[cameron-eyds]

Moving into staging for my own tracking until commit is ready, but have fixed locally.
Will bundle fix with ticket 18197.

[tlebedovich]

@cameron-eyds great work. moving to RFQA.
@chdivyareddy - I only tested with Staff, please also check for Qualified Supplier thanks.

[chdivyareddy]

Verified in DEV for Staff & QS flows.

Staff:

QS: