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

Integrate Flipper in RNTester on Android #27631

Closed
wants to merge 5 commits into from

Conversation

safaiyeh
Copy link
Contributor

@safaiyeh safaiyeh commented Dec 29, 2019

Summary

0.62 Flipper Support items: #27565
Made RNTester's Android Flipper implementation consistent with the template.
Added Flipper to the iOS build.
Screen Shot 2019-12-28 at 7 06 42 PM

Changelog

[Internal] [Added] - RNTester Android Fipper updates

Test Plan

Run RNTester and see if it connects to Flipper.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 29, 2019
@safaiyeh safaiyeh mentioned this pull request Dec 29, 2019
30 tasks
@react-native-bot react-native-bot added Platform: Android Android applications. Platform: iOS iOS applications. Type: Enhancement A new feature or enhancement of an existing feature. labels Dec 29, 2019
RNTester/Podfile Outdated Show resolved Hide resolved
@rickhanlonii
Copy link
Member

Could you rebase and take a look at the failures?

@rickhanlonii
Copy link
Member

I rebased onto master and tested iOS

Screen Shot 2020-01-10 at 11 08 37 AM

@rickhanlonii
Copy link
Member

Seems like the failing tests are failing on master as well so unrelated

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@rickhanlonii has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Member

@passy passy left a comment

Choose a reason for hiding this comment

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

Android side LGTM!

@safaiyeh safaiyeh changed the title [RNTester][Flipper] Make Android consistent with 0.62 template. Add Flipper to iOS [RNTester][Flipper] Make Android consistent with 0.62 template Jan 15, 2020
@safaiyeh
Copy link
Contributor Author

Reverted iOS changes in favor of #27762, @priteshrnandgaonkar solved the use_frameworks! issue.

This just makes the updates to Android to be consistent with the template app

cc @rickhanlonii

@rickhanlonii rickhanlonii changed the title [RNTester][Flipper] Make Android consistent with 0.62 template Integrate Flipper in RNTester on Android Jan 17, 2020
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@rickhanlonii has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@passy has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @safaiyeh in 6579d8f.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Jan 29, 2020
@safaiyeh safaiyeh deleted the flipper-RNTester branch January 29, 2020 16:08
rickhanlonii pushed a commit that referenced this pull request Feb 5, 2020
Summary:
0.62 Flipper Support items: #27565
Made RNTester's Android Flipper implementation consistent with the template.
~~Added Flipper to the iOS build.~~
<img width="1259" alt="Screen Shot 2019-12-28 at 7 06 42 PM" src="https://user-images.githubusercontent.com/8675043/71551835-37290800-29a5-11ea-9eac-b119a69a68c1.png">

## Changelog

[Internal] [Added] - RNTester Android Fipper updates
Pull Request resolved: #27631

Test Plan: Run RNTester and see if it connects to Flipper.

Reviewed By: rickhanlonii

Differential Revision: D19345093

Pulled By: passy

fbshipit-source-id: 6957c1ca3f4a5bb7f0e581c5daf8ddeac5d87eea
osdnk pushed a commit to osdnk/react-native that referenced this pull request Mar 9, 2020
Summary:
0.62 Flipper Support items: facebook#27565
Made RNTester's Android Flipper implementation consistent with the template.
~~Added Flipper to the iOS build.~~
<img width="1259" alt="Screen Shot 2019-12-28 at 7 06 42 PM" src="https://user-images.githubusercontent.com/8675043/71551835-37290800-29a5-11ea-9eac-b119a69a68c1.png">

## Changelog

[Internal] [Added] - RNTester Android Fipper updates
Pull Request resolved: facebook#27631

Test Plan: Run RNTester and see if it connects to Flipper.

Reviewed By: rickhanlonii

Differential Revision: D19345093

Pulled By: passy

fbshipit-source-id: 6957c1ca3f4a5bb7f0e581c5daf8ddeac5d87eea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: Android Android applications. Platform: iOS iOS applications. Type: Enhancement A new feature or enhancement of an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants