Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add COMPOSITION in KeyPath to target root composition layer. #1559

Merged
merged 6 commits into from
May 29, 2020

Conversation

boduan-coder
Copy link
Contributor

No description provided.

issue-repro/src/main/AndroidManifest.xml Outdated Show resolved Hide resolved
@gpeal
Copy link
Collaborator

gpeal commented May 10, 2020

@boduan-coder Can you add some javadocs? Other than that, lgtm!

@boduan-coder boduan-coder changed the title [WIP]Add COMPOSITION in KeyPath. Add COMPOSITION in KeyPath to target root composition layer. May 10, 2020
@boduan-coder boduan-coder requested a review from gpeal May 10, 2020 02:11
@boduan-coder
Copy link
Contributor Author

@gpeal , is the screenshot test automatically triggered? I can't see the test report.

@gpeal
Copy link
Collaborator

gpeal commented May 12, 2020

@boduan-coder I accidentally broke them but just fixed them here #1563

Could you merge then push to kick it off again?

@LottieSnapshotBot
Copy link

Snapshot Tests
28: Report Diff

Copy link
Collaborator

@gpeal gpeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this!

lottie/src/main/java/com/airbnb/lottie/model/KeyPath.java Outdated Show resolved Hide resolved
@LottieSnapshotBot
Copy link

Snapshot Tests
28: Report Diff

@boduan-coder
Copy link
Contributor Author

@gpeal , could you please help merge this PL if everything looks good?

@boduan-coder
Copy link
Contributor Author

Friendly ping :)

Copy link
Collaborator

@gpeal gpeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this @boduan-coder and sorry for the delay. I have been hard at work on MvRx 2.0 and that has been eating up all of my limited open source nights and weekends time
https://github.com/airbnb/MvRx/commits/release/2.0.0

@gpeal gpeal merged commit b7bd15c into airbnb:master May 29, 2020
boduan-coder added a commit to boduan-coder/lottie-android that referenced this pull request Jun 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants