Codepush HOC should unmount app before restart

[savv]

RN Apps currently don't get the opportunity to destruct themselves prior to a restart, leading to crashes and bugs:

#2567 (comment)

This is an attempt to mitigate these crashes, by properly unmounting <App />.

I'm looking forward to your feedback

[savv]

@microsoft-github-policy-service agree

[savv]

ping?

[viniciusmoreria]

this is not missing the state in the class construtor?

      constructor() {
        super();
        this.state = { inactive: false };
      }

[velimir-jankovic]

@savv Thank you for this contribution! I'm a bit concerned that we may affect existing apps with this change.
I think it would be a good idea to put this change behind an option (for now), what do you think?

[DmitriyKirakosyan]

Closing as stale.