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

[#165661178] Removed all references to RCTGeolocation #1026

Merged
merged 2 commits into from
May 8, 2019

Conversation

Edigio
Copy link
Contributor

@Edigio Edigio commented May 8, 2019

Removed all references to RCTGeolocation

Removed all references to RCTGeolocation
@digitalcitizenship
Copy link

digitalcitizenship commented May 8, 2019

Affected stories

  • 🐞 #165661178: App Store Connect - Missing Purpose String in Info.plist

Generated by 🚫 dangerJS

@Edigio
Copy link
Contributor Author

Edigio commented May 8, 2019

From this post I think I have found the solution for this issue: facebook/react-native#20879 (comment)

@codecov
Copy link

codecov bot commented May 8, 2019

Codecov Report

Merging #1026 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1026      +/-   ##
=========================================
+ Coverage   46.14%   46.2%   +0.06%     
=========================================
  Files         204     204              
  Lines        4575    4560      -15     
  Branches      893     862      -31     
=========================================
- Hits         2111    2107       -4     
+ Misses       2457    2446      -11     
  Partials        7       7

@fpersico fpersico merged commit acfe238 into master May 8, 2019
@fabriziofff fabriziofff deleted the 165661178-purpose-string-infoplist-issue branch January 21, 2022 13:35
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.

4 participants