proposal to detach children before self
This pull request stems from https://github.com/uber/RIBs/issues/589 and attempt to swap the detachment order between self and any attached children to avoid a small window in which the child is active and parent is not.
Note that this only updates the Android part - the corresponding iOS behavior remains "detach self before children" even after this pull request.
Has this been verified in the Uber monorepo? Tests passed and apps worked?
Has this been verified in the Uber monorepo? Tests passed and apps worked?
No, not yet - was creating this pull request to collect information on whether to do this at all (and if yes, gather information on what would be considered a sufficient test)
Suggest a first pass with a custom built dependency to understand the scope of the migration
Update: After smoke testing with Uber's big apps, decided to limit the change to only move the "detach self" ahead of "detach children" and leave the "will detach" position untouched