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

Bug - Transfer Due to Sale/Gift - Issue when changing from SO to TIC #18197

Closed
tlebedovich opened this issue Oct 17, 2023 · 12 comments
Closed

Bug - Transfer Due to Sale/Gift - Issue when changing from SO to TIC #18197

tlebedovich opened this issue Oct 17, 2023 · 12 comments
Assignees
Labels
Assets bug Something isn't working UX Assurance

Comments

@tlebedovich
Copy link

tlebedovich commented Oct 17, 2023

This bug is most likely also related to bug #17754


on DEV site:
The order you do this is important:

IF you open an MHR with a Sole Owner
THEN add a new owner to group 2 to make this a tenants in Common
THEN update the interest for Group 1 which has the original owner (because Interest is red N/A)
THEN add another new owner to Group 1 (the group with the original owner)
THEN delete the original owner from Group 1 (which moves into a previous owner group as it should)

The Issue that appears: Now that Group 1 no longer has any original owners and just has a new owner in it, the badge for Group 1 should say ADDED and the Button should be Delete Group (just like in Group 2)

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.


screenshots (in order)

  1. added new owner to group 2:

Screen Shot 2023-10-16 at 5 31 38 PM

  1. updated the interest for group 1:

Screen Shot 2023-10-16 at 5 31 53 PM

  1. added a another new owner to group 1:
    Screen Shot 2023-10-16 at 5 33 04 PM

  2. deleted the original owner from group 1:
    Screen Shot 2023-10-16 at 5 33 44 PM

@dimak1
Copy link
Collaborator

dimak1 commented Dec 7, 2023

You can re-assign this to yourself @cameron-eyds

@tlebedovich
Copy link
Author

@cameron-eyds looks good to me. @chdivyareddy I tested logged in as staff, please also test logged in as a Qualified Supplier. Thanks. Moving to RFQA

@chdivyareddy
Copy link
Collaborator

Verified in DEV for both staff & QS flow.

Staff:
image.png

QS:
image.png

@chdivyareddy
Copy link
Collaborator

@cameron-eyds , I'm now not seeing the 'Added' badge beside the Group1 (this change was made while back before the Vue3 upgrade I believe). Please take a look, thanks!!

Screenshot 2024-01-12 at 3.46.44 PM.png

Expected:
image.png

@cameron-eyds
Copy link
Collaborator

Hmm If this was verified in DEV, Dec 12, that would have been the Vue3 build.
I will have a look!

@cameron-eyds
Copy link
Collaborator

cameron-eyds commented Jan 15, 2024

I followed the steps in the ticket in DEV and saw the expected result:
Screenshot 2024-01-15 at 7.33.10 AM.png

Confirmed in TEST as well.
Screenshot 2024-01-15 at 7.44.10 AM.png

@chdivyareddy Perhaps there was a different order of operations to reproduce, outside the steps Tracey had outlined above?

*Edit: I found a scenario, where if you don't edit the group Interest values and add a new owner to the group, which requires interest values and delete the original owner, it won't show the added.

That being said, it was not the exact steps outlined in the ticket, but potentially we still want it to say added?
I'm not 100% sure if it's the same requirement but should we add it in for this scenario as well?

@chdivyareddy
Copy link
Collaborator

@cameron-eyds , Yes you are right! 'Added' badge doesn't gets added when the interest value is not edited before adding the owner to the existing group. I think we still would like to display the 'Added' badge in this scenario as well.
cc @tlebedovich Can you please take a look, thanks!!

image.png

@tlebedovich
Copy link
Author

having a look now...

@tlebedovich
Copy link
Author

@cameron-eyds @chdivyareddy - I am in agreement. Since Group 1 no longer contains any of the original existing owners (original SO was deleted), Group 1 would be considered a net new group, so the ADDED badge would appear in this scenario as well.

@cameron-eyds
Copy link
Collaborator

@tlebedovich @chdivyareddy Thank you both for checking/confirming! Will update this

@cameron-eyds
Copy link
Collaborator

@chdivyareddy Have updated as per discussions above, ready for another look!

@chdivyareddy
Copy link
Collaborator

Verified in DEV for both scenarios.

image.png

image.png

@PCC199 PCC199 closed this as completed Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Assets bug Something isn't working UX Assurance
Projects
None yet
Development

No branches or pull requests

5 participants