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

Update README.md #962

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Update README.md #962

merged 1 commit into from
Mar 12, 2024

Conversation

DivyaniPatel96
Copy link
Contributor

Update readme file

Update readme file
@DivyaniPatel96 DivyaniPatel96 requested a review from Akhror March 8, 2024 14:13
@@ -83,14 +87,20 @@ git clone https://github.com/UNICEFECAR/parenting-app-bebbo-mobile.git
```
2. Install NPM packages (DO NOT USE yarn!)
```sh
npm install --legacy-peer-deps
npm install
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you test this branch to make sure this is not a concern anymore?

@@ -37,13 +37,17 @@ The app can also operate in an offline mode in environments with limited interne
## Technology stack

* [React Native](https://reactnative.dev/)
* Project is set with Latest version of react native:
* React native - 0.72.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's please change above two lines to "The project is configured to utilize React Native version 0.72.5."

@Akhror
Copy link
Collaborator

Akhror commented Mar 9, 2024

We should merge these changes to Development branch instead.

```
3. Only on Mac, go to "ios" folder, and run
```sh
pod install
```
Before running iOS App on Device, necessary certificate setup from Apple developer account is required.
After each time pod install, copy [CP-User] [RNFB] Core Configuration && [CP-User] [RNFB] Crashlytics Configuration in each target’s build phases if missing from main target (ParentBuddyApp)
After each time pod install, copy [CP-User] [RNFB] Core Configuration && [CP-User] [RNFB] Crashlytics Configuration in each target’s build phases if missing from main target (ParentBuddyApp). In Project use below targets.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Before running the iOS app on a device, it's essential to set up the necessary certificates from the Apple Developer account. After each pod install, ensure to copy the [CP-User] [RNFB] Core Configuration and [CP-User] [RNFB] Crashlytics Configuration into each target's build phases if they are missing from the main target (ParentBuddyApp). Below are the project targets to use:

1. ParentBuddyApp - Bebbo Production target
2. ParentBuddyAppDev - Bebbo Development target
3. ParentBuddyAppXk - Foleja Production target
4. ParentBuddyAppXkDev - Foleja Development target
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's use a, b, c, d instead of 1, 2, 3, 4

@@ -199,6 +211,49 @@ Refer this library for custom fonts: https://github.com/oblador/react-native-vec

* In tsconfig.json keep dynamicImportsClass value to ./app/bebbo/* as follows :
Copy link
Collaborator

Choose a reason for hiding this comment

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

In tsconfig.json, set the dynamicImportsClass value to ./app/bebbo/*:

export const buildFor = buildForFoleja;
```

* In tsconfig.json keep dynamicImportsClass value to ./app/xk/* as follows :
Copy link
Collaborator

Choose a reason for hiding this comment

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

In tsconfig.json, set the dynamicImportsClass value to ./app/xk/*:

@DivyaniPatel96 DivyaniPatel96 merged commit a228efd into main Mar 12, 2024
2 checks passed
deepakmali-dgsl pushed a commit that referenced this pull request Aug 13, 2024
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.

2 participants