added onComplete callback to animation config

[vaukalak]

This fix provides possibility to subscribe to a child animation lifecycle. You'll be able to observe every single animation:

Animated.sequence([
            Animated.timing(
                this.state.scale,
                {
                    toValue: 0,
                    duration: 300,
                    onComplete: () => this.setState({someProp: 'new value'})
                }
            ),
            Animated.timing(
                this.state.scale,
                {
                    toValue: 1,
                    duration: 300
                }
            ),
        ]).start();

state.someProp, will updated with 'new value' when the first animation will be completed.

[ghost]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

[eslint-bot]

keyword-spacing: Expected space(s) after "if".

[ghost]

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

[vaukalak]

I'm sorry, that's my first PR to react-native, and I'm a bit confused.. I've committed changes required by @eslint-bot . If I understand correctly a new build was not triggered, So I don't have an idea what to do now, to make that proceed.

[vadzim]

+1

[ghost]

@vaukalak I tried to find reviewers for this pull request and wanted to ping them to take another look. However, based on the blame information for the files in this pull request I couldn't find any reviewers. This sometimes happens when the files in the pull request are new or don't exist on master anymore. Is this pull request still relevant? If yes could you please rebase? In case you know who has context on this code feel free to mention them in a comment (one person is fine). Thanks for reading and hope you will continue contributing to the project.

[ghost]

@vaukalak updated the pull request - view changes

[ghost]

@vaukalak updated the pull request - view changes

[vaukalak]

@foghina in base on history, I can find, that you did a lot of contributions in this file. Could you please review this?

[foghina]

LGTM, will fix a couple of nits before landing.

@facebook-github-bot import

[ghost]

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.

[vaukalak]

@foghina I'm sorry, couldn't understand your comment. Could you please tell me what to do, to make that proceed?

[foghina]

@vaukalak I just wanted to make a tiny fix (rename enhanceCallback to _combineCallbacks), didn't want to waste a round trip (have you update the PR), so I used import instead of shipit so I could make the change internally.

Your PR is landing as we speak, no need to do anything :)