Skip to content
This repository has been archived by the owner on Jun 16, 2023. It is now read-only.

fix status for non authorized/pending camera #1547

Merged
merged 3 commits into from
May 7, 2018
Merged

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented May 7, 2018

No description provided.

Copy link
Collaborator

@sibelius sibelius left a comment

Choose a reason for hiding this comment

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

can we use early return here?

I think it is better to understand

can we also use CONST instead of pure strings?

const STATUS = {
READY: 'READY',
}

@n1ru4l
Copy link
Collaborator Author

n1ru4l commented May 7, 2018

@sibelius Done. Please feel free to merge after CI passes 😊

@@ -102,6 +102,12 @@ type StateType = {

type Status = 'READY' | 'PENDING_AUTHORIZATION' | 'NOT_AUTHORIZED';

const CameraStatus = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

export this so end user can use it as well

Copy link
Collaborator Author

@n1ru4l n1ru4l May 7, 2018

Choose a reason for hiding this comment

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

It is exported on RNCamera.Constants

@n1ru4l
Copy link
Collaborator Author

n1ru4l commented May 7, 2018

Please not merge yet. I think I made a mistake

@jgfidelis
Copy link
Collaborator

I was going to comment that naming was not making much sense hehe. Can we merge it?

@n1ru4l
Copy link
Collaborator Author

n1ru4l commented May 7, 2018

@jgfidelis Feel free to merge 😉

@sibelius sibelius merged commit 86125a0 into master May 7, 2018
Fconstant pushed a commit that referenced this pull request May 7, 2018
Fconstant pushed a commit that referenced this pull request May 7, 2018
Fconstant added a commit that referenced this pull request May 7, 2018
timxyz pushed a commit to 3sidedcube/react-native-camera that referenced this pull request May 15, 2018
* master: (355 commits)
  UPDATED renderChildren to return this.props.children instead of null
  feat(types): add types for react-native-camera#1547 (react-native-camera#1548)
  fix(rn-camera): inject correct status
  feat(rn-camera): use and export constants
  fix status for non authorized/pending camera
  The correct capture preset for photos is AVCaptureSessionPresetPhoto
  add gradle 3 on issue template
  feat(types): add types for react-native-camera#1523
  feat(types): add types for react-native-camera#1518 (FaCC)
  Add missing infer annotation dependency.
  fix(cache): store video recordings in same directory as photos
  fix(rn_camera): improve naming
  feat(rn_camera): add function as children
  1.1.2
  build(change-log): v1.1.2-4
  FIXED warning for focusDepth on front-facing camera
  chore(cameraview): integrate google's cameraview directly on rncamera?
  fix(search-paths): remove unnecessary search paths and add missing one
  1.1.1
  build(change-log): v1.1.1-3
  ...

# Conflicts:
#	ios/RCTCameraManager.m
rad182 added a commit to rad182/react-native-camera that referenced this pull request May 18, 2018
* upstream/master: (23 commits)
  Updates Pod Install Instructions
  reenable camera on android emulator
  Create GradleUpgradeGuide.md
  Android: Fix java.lang.ArrayIndexOutOfBoundsException with image rotation
  Fix run time error missing facebook infer annotation
  UPDATED renderChildren to return this.props.children instead of null
  feat(types): add types for react-native-camera#1547 (react-native-camera#1548)
  fix(rn-camera): inject correct status
  feat(rn-camera): use and export constants
  fix status for non authorized/pending camera
  The correct capture preset for photos is AVCaptureSessionPresetPhoto
  add gradle 3 on issue template
  feat(types): add types for react-native-camera#1523
  feat(types): add types for react-native-camera#1518 (FaCC)
  Add missing infer annotation dependency.
  fix(cache): store video recordings in same directory as photos
  fix(rn_camera): improve naming
  feat(rn_camera): add function as children
  1.1.2
  build(change-log): v1.1.2-4
  ...
@sibelius sibelius deleted the n1ru4l-patch-1 branch June 14, 2018 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants