dispatch: return a promise resolving on navigation animation completion about rfcs

[slorber]

this works fine:

goBackAsync = () => {
  const promise = new Promise(resolve => {
    const subscription = this.props.navigation.addListener('didBlur', () => {
      subscription.remove();
      resolve();
    });
  });
  this.props.navigation.goBack();
  return promise;
};

[brentvatne]

you could wrap dispatch with a fn that adds a listener for didBlur and then resolves the promise and removes itself

[dantman]

The dispatch API is based on Redux. And When Redux is React Navigation's dispatch method is 100% replaced with the standard Redux dispatch method. So we cannot change the dispatch API to return a Promise.

Something similar to React Native's InteractionManager.runAfterInteractions but for transitions would probably work better.

I think I had more concrete API ideas in one of my comments on another issue.

[slorber]

Thanks will try to use did Blur. With Redux you can easily return a promise from dispatch and it's actually common to do so, for example in thunks. Nothing forbids a Middleware to return something useful. But I agree with the idea of a hook to wait for navigation completion. But maybe a singleton would not be appropriate and did Blur is already that? Le 9 févr. 2018 06:09, "Daniel Friesen" <[email protected]> a écrit : The dispatch API is based on Redux. And When Redux is React Navigation's dispatch method is 100% replaced with the standard Redux dispatch method. So we cannot change the dispatch API to return a Promise. Something similar to React Native's InteractionManager.runAfterInteractions but for transitions would probably work better. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAtvPnGLmyT637txluili6l-6wUcA-fyks5tS9MRgaJpZM4R9BSN> .

[dantman]

@slorber Oh, I'm still pre-middleware. Didn't see that things changed.

[slorber]

didBlur seems to work fine @brentvatne thanks

I've made a pr to the doc as it's not well documented: react-navigation/react-navigation.github.io#54

[brentvatne]

cool, closing this then!