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

Chore: Ejecting from expo managed #134

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Conversation

chrysloi
Copy link
Contributor

@chrysloi chrysloi commented Jul 5, 2024

Chore: Ejecting from expo managed

Description

This is to eject from expo managed to enable us to run the app with the webrtc feature.

Steps to test

Android

  • Check if you have JDK correctly installed from terminal by running:
java --version
  • If it returns like below or related you can go to next step.
    Otherwise find ways to configure JDK on the current os you're using.

⚠️ NB: it may vary according to OS or JDK versions

openjdk version "17.0.10" 2024-01-16 LTS
OpenJDK Runtime Environment Zulu17.48+15-CA (build 17.0.10+7-LTS)
OpenJDK 64-Bit Server VM Zulu17.48+15-CA (build 17.0.10+7-LTS, mixed mode, sharing)

  • Clone this repo and checkout to ch-ejecting-expo-managed
git clone [email protected]:atlp-rwanda/commanders-rn-medica.git
cd commanders-rn-medica
git checkout ch-ejecting-expo-managed
npm install
  • Now run the app
npm run android

IOS

  • Clone this repo and checkout to ch-ejecting-expo-managed
git clone [email protected]:atlp-rwanda/commanders-rn-medica.git
cd commanders-rn-medica
git checkout ch-ejecting-expo-managed
npm install
  • install pods
cd ./ios
pod install
cd ../
  • Now run the app
npm run ios

⚠️ NB:

  • If you're running ios and you have xcode less than 13.x.x it'll not work
  • if you miss a step the next might not work

@chrysloi chrysloi added Chore Define what a chore is Need Review labels Jul 5, 2024
@chrysloi chrysloi self-assigned this Jul 5, 2024
@Latiah
Copy link
Contributor

Latiah commented Jul 5, 2024

hey @chrysloi I faced this while running your pr
Screenshot 2024-07-05 at 16 30 43

@chrysloi
Copy link
Contributor Author

chrysloi commented Jul 5, 2024

hey @chrysloi I faced this while running your pr Screenshot 2024-07-05 at 16 30 43

this seems to be compatibility issues with you JDK version please share what you get when you run java --version

also share the output you get after running npx react-native doctor

@Latiah
Copy link
Contributor

Latiah commented Jul 5, 2024

Screenshot 2024-07-05 at 18 51 03

@chrysloi
Copy link
Contributor Author

chrysloi commented Jul 5, 2024

Screenshot 2024-07-05 at 18 51 03

@Latiah you'll need to install JDK 17 that's the suggested one as in the last versions of JDK are not supported with some of the packages. Make sure you did set the ANDROID_HOME correctly as the react-native doctor isn't finding it!

Copy link
Contributor

@kibongos40 kibongos40 left a comment

Choose a reason for hiding this comment

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

I'm getting this warning.

More error details at the end down

image

If I dismiss it, the app runs well no problem

simulator_screenshot_77B78B6D-19A1-41C2-90CF-612DA19AA957

If you want to change the appearance of status bar, you have to change UIViewControllerBasedStatusBarAppearance key in the Info.plist to YES

+[RNSScreenWindowTraits assertViewControllerBasedStatusBarAppearenceSet]
RNSScreenWindowTraits.mm:18
-[RNSScreenView setStatusBarStyle:]
__49-[RCTComponentData createPropBlock:isShadowView:]_block_invoke_10
__49-[RCTComponentData createPropBlock:isShadowView:]_block_invoke_2.110
__49-[RCTComponentData propBlockForKey:isShadowView:]_block_invoke_2
RCTPerformBlockWithLogFunction
RCTPerformBlockWithLogPrefix
__49-[RCTComponentData propBlockForKey:isShadowView:]_block_invoke
__50-[RCTComponentData setProps:forView:isShadowView:]_block_invoke
NSDICTIONARY_IS_CALLING_OUT_TO_A_BLOCK
-[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:]
-[RCTComponentData setProps:forView:isShadowView:]
-[RCTComponentData setProps:forView:]
__50-[RCTUIManager createView:viewName:rootTag:props:]_block_invoke.415
__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke
__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke.484
__RCTExecuteOnMainQueue_block_invoke
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_main_queue_drain
_dispatch_main_queue_callback_4CF
CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE
__CFRunLoopRun
CFRunLoopRunSpecific
GSEventRunModal
-[UIApplication _run]
UIApplicationMain
main
start_sim
0x0

@Latiah
Copy link
Contributor

Latiah commented Jul 8, 2024

Screenshot 2024-07-05 at 18 51 03

@Latiah you'll need to install JDK 17 that's the suggested one as in the last versions of JDK are not supported with some of the packages. Make sure you did set the ANDROID_HOME correctly as the react-native doctor isn't finding it!

Thanks .After setting up everything correctly I was able to run the app everything is fine I encountered 0 errors

@chrysloi chrysloi force-pushed the ch-ejecting-expo-managed branch from 3ce4512 to 710dc0b Compare July 9, 2024 07:32
@chrysloi
Copy link
Contributor Author

chrysloi commented Jul 9, 2024

I'm getting this warning.

More error details at the end down

image

If I dismiss it, the app runs well no problem

simulator_screenshot_77B78B6D-19A1-41C2-90CF-612DA19AA957

If you want to change the appearance of status bar, you have to change UIViewControllerBasedStatusBarAppearance key in the Info.plist to YES

+[RNSScreenWindowTraits assertViewControllerBasedStatusBarAppearenceSet] RNSScreenWindowTraits.mm:18 -[RNSScreenView setStatusBarStyle:] __49-[RCTComponentData createPropBlock:isShadowView:]_block_invoke_10 __49-[RCTComponentData createPropBlock:isShadowView:]_block_invoke_2.110 __49-[RCTComponentData propBlockForKey:isShadowView:]_block_invoke_2 RCTPerformBlockWithLogFunction RCTPerformBlockWithLogPrefix __49-[RCTComponentData propBlockForKey:isShadowView:]_block_invoke __50-[RCTComponentData setProps:forView:isShadowView:]_block_invoke NSDICTIONARY_IS_CALLING_OUT_TO_A_BLOCK -[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:] -[RCTComponentData setProps:forView:isShadowView:] -[RCTComponentData setProps:forView:] __50-[RCTUIManager createView:viewName:rootTag:props:]_block_invoke.415 __44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke __44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke.484 __RCTExecuteOnMainQueue_block_invoke _dispatch_call_block_and_release _dispatch_client_callout _dispatch_main_queue_drain _dispatch_main_queue_callback_4CF CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE __CFRunLoopRun CFRunLoopRunSpecific GSEventRunModal -[UIApplication _run] UIApplicationMain main start_sim 0x0

@kibongos40 That's addressed, Please check again!

@chrysloi chrysloi requested a review from kibongos40 July 9, 2024 07:39
@chrysloi chrysloi force-pushed the ch-ejecting-expo-managed branch from 710dc0b to 863cd6b Compare July 9, 2024 15:29
@911samuel
Copy link
Contributor

Screenshot from 2024-07-10 10-24-59

it it failing on my device

Copy link
Contributor

@fredshema fredshema left a comment

Choose a reason for hiding this comment

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

The app is starting correctly on android. Nice work👍🏽

@Emmyfrank
Copy link
Contributor

The app is starting correctly on android. Nice work👍🏽

ok which is good

@chrysloi chrysloi force-pushed the ch-ejecting-expo-managed branch from 863cd6b to 9787eb5 Compare July 12, 2024 13:13
Copy link

gitguardian bot commented Jul 12, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@chrysloi chrysloi force-pushed the ch-ejecting-expo-managed branch 2 times, most recently from 9787eb5 to 67cdf52 Compare July 12, 2024 14:26
@chrysloi chrysloi force-pushed the ch-ejecting-expo-managed branch from 67cdf52 to ee3f18a Compare July 12, 2024 14:28
Copy link
Contributor

@911samuel 911samuel left a comment

Choose a reason for hiding this comment

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

Great work! issue was on my android studion sdk now resolved

Copy link
Contributor

@kibongos40 kibongos40 left a comment

Choose a reason for hiding this comment

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

Now it works
simulator_screenshot_B3F6B755-040C-4B38-93C7-D4257B010B68

@chrysloi chrysloi added need aprove Improvements or additions to documentation and removed Need Review need aprove Improvements or additions to documentation labels Jul 22, 2024
Copy link
Contributor

@mukunzidd mukunzidd left a comment

Choose a reason for hiding this comment

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

LGTM

@mukunzidd mukunzidd merged commit 92ea656 into dev Jul 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore Define what a chore is need aprove Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants