fixed: delete attribute even if related collection does not exist - #6045
Akshay-Rana-Gujjar wants to merge 13 commits into
Conversation
fanatic75
left a comment
There was a problem hiding this comment.
@Akshay-Rana-Gujjar , Let's add a test case for this scenario
|
Hey @fanatic75, I have added the test case, please review 🙏 |
|
Hey @fanatic75 , any update on this? |
|
@fanatic75 please review 🙏 |
|
Hey @stnguyen90, Could you please help to review this PR. |
fanatic75
left a comment
There was a problem hiding this comment.
Just fix these small changes
Otherwise looks good to me. Good work!
|
@abnegate please help review this PR, thanks |
|
@stnguyen90 Can you please help with this merge it is almost 1 month and It is open 😢 |
|
Hey @Haimantika & @tessamero, could you please help merge this MR? It has been open for quite some time. |
abnegate
left a comment
There was a problem hiding this comment.
How do you get to the state where the related collection doesn't exist but the relationship does? Are there steps to reproduce?
This sounds like the bug might be related to deleting a collection itself (which should delete all of it's relationships)
Yes, I followed the steps mentioned in this issue. 👟 Reproduction steps
|
|
I agree, we should not get to this point we have a hanging attribute, but at least we should be able to fix it if it happens |
Hey @fogelito, could you please explain what I need to do further to get approvals and merge this MR? Thanks. |
|
Hey, thanks a lot 🙏 , @abnegate can you give your review here, please? |
|
@Akshay-Rana-Gujjar The root cause of the issue is that when a collection is deleted, the relationships are not deleted from the |
|
Closing as we've updated the logic to delete the related attributes when a collection is deleted. |
What does this PR do?
Delete attribute even if related collection does not exist.
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)
Related PRs and Issues
Checklist