Skip to content

80 test driver app chat camera support - #163

Open
dimonco wants to merge 54 commits into
masterfrom
80_testDriverAppChatCameraSupport
Open

80 test driver app chat camera support#163
dimonco wants to merge 54 commits into
masterfrom
80_testDriverAppChatCameraSupport

Conversation

@dimonco

@dimonco dimonco commented Jul 9, 2018

Copy link
Copy Markdown
Contributor

No description provided.

tanjabatz and others added 30 commits June 24, 2018 20:24
It's needed in order for the Camera component to work
@codecov

codecov Bot commented Jul 9, 2018

Copy link
Copy Markdown

Codecov Report

Merging #163 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #163   +/-   ##
=======================================
  Coverage   75.42%   75.42%           
=======================================
  Files           7        7           
  Lines        1078     1078           
  Branches      251      251           
=======================================
  Hits          813      813           
  Misses         93       93           
  Partials      172      172

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb55cc8...e26805f. Read the comment docs.

@markrudtke

Copy link
Copy Markdown
Contributor

before you merge, please test on IOS if everything builds properly

@longpdo

longpdo commented Jul 9, 2018

Copy link
Copy Markdown
Member

without npm start and expo we can't show ios support on the demo day, since apk can't be installed on ios.

@dimonco

dimonco commented Jul 9, 2018

Copy link
Copy Markdown
Contributor Author

It is possible to run react-native run-ios, but I cannot test this because I don't have an iOS env

@davidhaller

Copy link
Copy Markdown

Cannot be merged into master because it breaks iOS/Expo build.

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.

5 participants