Skip to content

Commit

Permalink
fix merge conflict
Browse files Browse the repository at this point in the history
  • Loading branch information
akinwale committed Aug 29, 2023
2 parents 998e56c + d84006f commit fd28e88
Show file tree
Hide file tree
Showing 127 changed files with 6,506 additions and 9,048 deletions.
4 changes: 2 additions & 2 deletions android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
multiDexEnabled rootProject.ext.multiDexEnabled
versionCode 1001035704
versionName "1.3.57-4"
versionCode 1001035801
versionName "1.3.58-1"
}

flavorDimensions "default"
Expand Down
4 changes: 2 additions & 2 deletions assets/images/new-expensify-dev.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions contributingGuides/PROPTYPES_CONVERSION_TABLE.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ const propTypes = {
};

// After
type Props = {
type ComponentProps = {
isVisible: boolean;
// Consider it as required unless you have proof that it is indeed an optional prop.
confirmText: string; // vs. confirmText?: string;
Expand Down Expand Up @@ -115,7 +115,7 @@ type Input = {

type Size = "small" | "medium" | "large";

type Props = {
type ComponentProps = {
unknownData: string[];

// It's not possible to infer the data as it can be anything because of reasons X, Y and Z.
Expand Down
22 changes: 22 additions & 0 deletions contributingGuides/TS_STYLE.md
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,28 @@ type Foo = {
type Color = "red" | "blue" | "green";
```

- Use `{ComponentName}Props` pattern for prop types.

```ts
// BAD
type Props = {
// component's props
};
function MyComponent({}: Props) {
// component's code
}
// GOOD
type MyComponentProps = {
// component's props
};
function MyComponent({}: MyComponentProps) {
// component's code
}
```

- For generic type parameters, use `T` if you have only one type parameter. Don't use the `T`, `U`, `V`... sequence. Make type parameter names descriptive, each prefixed with `T`.

> Prefix each type parameter name to distinguish them from other types.
Expand Down
4 changes: 2 additions & 2 deletions ios/NewExpensify/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
<key>CFBundlePackageType</key>
<string>APPL</string>
<key>CFBundleShortVersionString</key>
<string>1.3.57</string>
<string>1.3.58</string>
<key>CFBundleSignature</key>
<string>????</string>
<key>CFBundleURLTypes</key>
Expand All @@ -40,7 +40,7 @@
</dict>
</array>
<key>CFBundleVersion</key>
<string>1.3.57.4</string>
<string>1.3.58.1</string>
<key>ITSAppUsesNonExemptEncryption</key>
<false/>
<key>LSApplicationQueriesSchemes</key>
Expand Down
4 changes: 2 additions & 2 deletions ios/NewExpensifyTests/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@
<key>CFBundlePackageType</key>
<string>BNDL</string>
<key>CFBundleShortVersionString</key>
<string>1.3.57</string>
<string>1.3.58</string>
<key>CFBundleSignature</key>
<string>????</string>
<key>CFBundleVersion</key>
<string>1.3.57.4</string>
<string>1.3.58.1</string>
</dict>
</plist>
Loading

0 comments on commit fd28e88

Please sign in to comment.