Skip to content

Commit 45c748d

Browse files
committed
Merge branch 'main' into @szymczak/create-buttons
2 parents 1420347 + 758691e commit 45c748d

367 files changed

Lines changed: 3489 additions & 2510 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/ISSUE_TEMPLATE/Accessibility.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,12 @@ What can we do to fix the issue?
3535
Check off any platforms that are affected by this issue
3636
--->
3737
Which of our officially supported platforms is this issue occurring on? Please only tick the box if you have provided a screen-recording in the thread for each platform:
38-
- [ ] Android: Native
38+
- [ ] Android: App
3939
- [ ] Android: mWeb Chrome
40-
- [ ] iOS: Native
40+
- [ ] iOS: App
4141
- [ ] iOS: mWeb Safari
42+
- [ ] iOS: mWeb Chrome
43+
- [ ] Windows: Chrome
4244
- [ ] MacOS: Chrome / Safari
4345
- [ ] MacOS: Desktop
4446

.github/ISSUE_TEMPLATE/Performance.md

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,11 @@ Note: These should be the same as the benchmarks collected before any changes.
2525

2626
## Platforms:
2727
Select the officially supported platforms where the issue was reproduced:
28-
- [ ] Android: Standalone
29-
- [ ] Android: HybridApp
28+
- [ ] Android: App
3029
- [ ] Android: mWeb Chrome
31-
- [ ] iOS: Standalone
32-
- [ ] iOS: HybridApp
30+
- [ ] iOS: App
3331
- [ ] iOS: mWeb Safari
32+
- [ ] iOS: mWeb Chrome
3433
- [ ] Windows: Chrome
3534
- [ ] MacOS: Chrome / Safari
3635
- [ ] MacOS: Desktop
@@ -39,12 +38,11 @@ Select the officially supported platforms where the issue was reproduced:
3938
<summary>Platforms Tested:</summary>
4039
On which of our officially supported platforms was this issue tested:
4140

42-
- [ ] Android: Standalone
43-
- [ ] Android: HybridApp
41+
- [ ] Android: App
4442
- [ ] Android: mWeb Chrome
45-
- [ ] iOS: Standalone
46-
- [ ] iOS: HybridApp
43+
- [ ] iOS: App
4744
- [ ] iOS: mWeb Safari
45+
- [ ] iOS: mWeb Chrome
4846
- [ ] Windows: Chrome
4947
- [ ] MacOS: Chrome / Safari
5048
- [ ] MacOS: Desktop

.github/ISSUE_TEMPLATE/Standard.md

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ ___
1010
**Version Number:**
1111
**Reproducible in staging?:**
1212
**Reproducible in production?:**
13-
**If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:**
1413
**If this was caught during regression testing, add the test name, ID and link from TestRail:**
1514
**Email or phone of affected tester (no customers):**
1615
**Logs:** https://stackoverflow.com/c/expensify/questions/4856
@@ -32,12 +31,11 @@ Can the user still use Expensify without this being fixed? Have you informed the
3231

3332
## Platforms:
3433
Select the officially supported platforms where the issue was reproduced:
35-
- [ ] Android: Standalone
36-
- [ ] Android: HybridApp
34+
- [ ] Android: App
3735
- [ ] Android: mWeb Chrome
38-
- [ ] iOS: Standalone
39-
- [ ] iOS: HybridApp
36+
- [ ] iOS: App
4037
- [ ] iOS: mWeb Safari
38+
- [ ] iOS: mWeb Chrome
4139
- [ ] Windows: Chrome
4240
- [ ] MacOS: Chrome / Safari
4341
- [ ] MacOS: Desktop
@@ -46,12 +44,11 @@ Select the officially supported platforms where the issue was reproduced:
4644
<summary>Platforms Tested:</summary>
4745
On which of our officially supported platforms was this issue tested:
4846

49-
- [ ] Android: Standalone
50-
- [ ] Android: HybridApp
47+
- [ ] Android: App
5148
- [ ] Android: mWeb Chrome
52-
- [ ] iOS: Standalone
53-
- [ ] iOS: HybridApp
49+
- [ ] iOS: App
5450
- [ ] iOS: mWeb Safari
51+
- [ ] iOS: mWeb Chrome
5552
- [ ] Windows: Chrome
5653
- [ ] MacOS: Chrome / Safari
5754
- [ ] MacOS: Desktop

.github/workflows/cherryPick.yml

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
workflow_dispatch:
55
inputs:
66
PULL_REQUEST_URL:
7-
description: The full URL of the Expensify/App pull request to cherry-pick
7+
description: The full URL of the E/App or E/Mobile-Expensify pull request to cherry-pick
88
required: true
99

1010
jobs:
@@ -43,9 +43,9 @@ jobs:
4343
submodules: true
4444
# Only fetch depth 0 for Mobile-Expensify, because it's a submodule and we need more history to cherry pick successfully
4545
fetch-depth: 0
46-
46+
4747
- name: Checkout staging branch without full history if cherry picking App
48-
if: ${{ steps.getPRInfo.outputs.REPO_FULL_NAME == 'Expensify/App' }}
48+
if: ${{ steps.getPRInfo.outputs.REPO_FULL_NAME == github.repository }}
4949
# v4
5050
uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608
5151
with:
@@ -77,21 +77,11 @@ jobs:
7777
SEMVER_LEVEL: "PATCH"
7878

7979
- name: Fetch history of relevant refs if cherry picking an App change
80-
if: ${{ steps.getPRInfo.outputs.REPO_FULL_NAME == 'Expensify/App' }}
80+
if: ${{ steps.getPRInfo.outputs.REPO_FULL_NAME == github.repository }}
8181
run: |
82-
# Temporary hack during transition when -staging suffix is being added to tags
83-
if git ls-remote origin refs/tags/${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }} | grep -q . ; then
84-
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}
85-
else
86-
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}-staging
87-
fi
82+
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}
8883
cd Mobile-Expensify
89-
# Temporary hack during transition when -staging suffix is being added to tags
90-
if git ls-remote origin refs/tags/${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }} | grep -q . ; then
91-
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}
92-
else
93-
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}-staging
94-
fi
84+
git fetch origin main staging --no-recurse-submodules --no-tags --shallow-exclude ${{ steps.getPreviousVersion.outputs.PREVIOUS_VERSION }}
9585
9686
- name: Get E/App version bump commit
9787
id: getVersionBumpCommit

.github/workflows/publishReactNativeAndroidArtifacts.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,14 +69,14 @@ jobs:
6969
- name: Check if patches changed
7070
id: didPatchesChange
7171
run: |
72-
if git diff --exit-code --name-only ${{ steps.getOldAppHash.outputs.HASH }}..${{ steps.getNewAppHash.outputs.HASH }} -- patches/react-native+*.patch patches/@react-native+*.patch; then
72+
if ! git diff --exit-code --name-only ${{ steps.getOldAppHash.outputs.HASH }}..${{ steps.getNewAppHash.outputs.HASH }} -- patches/react-native+*.patch patches/@react-native+*.patch; then
7373
echo "::notice::Detected changes in patches"
7474
echo "DID_PATCHES_CHANGE=true" >> "$GITHUB_OUTPUT"
7575
exit 0
7676
fi
7777
7878
if [[ '${{ matrix.is_hybrid }}' == 'true' ]]; then
79-
if git -C Mobile-Expensify diff --exit-code --name-only ${{ steps.getOldMobileExpensifyHash.outputs.HASH }}..${{ steps.getNewMobileExpensifyHash.outputs.HASH }} -- patches/react-native+*.patch patches/@react-native+*.patch; then
79+
if ! git -C Mobile-Expensify diff --exit-code --name-only ${{ steps.getOldMobileExpensifyHash.outputs.HASH }}..${{ steps.getNewMobileExpensifyHash.outputs.HASH }} -- patches/react-native+*.patch patches/@react-native+*.patch; then
8080
echo "::notice::Detected changes in patches"
8181
echo "DID_PATCHES_CHANGE=true" >> "$GITHUB_OUTPUT"
8282
exit 0

Mobile-Expensify

android/app/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@ android {
114114
minSdkVersion rootProject.ext.minSdkVersion
115115
targetSdkVersion rootProject.ext.targetSdkVersion
116116
multiDexEnabled rootProject.ext.multiDexEnabled
117-
versionCode 1009013204
118-
versionName "9.1.32-4"
117+
versionCode 1009013305
118+
versionName "9.1.33-5"
119119
// Supported language variants must be declared here to avoid from being removed during the compilation.
120120
// This also helps us to not include unnecessary language variants in the APK.
121121
resConfigs "en", "es"

contributingGuides/REVIEWER_CHECKLIST.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@
1111
- [ ] I included screenshots or videos for tests on [all platforms](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#make-sure-you-can-test-on-all-platforms)
1212
- [ ] I verified that the composer does not automatically focus or open the keyboard on mobile unless explicitly intended. This includes checking that returning the app from the background does not unexpectedly open the keyboard.
1313
- [ ] I verified tests pass on **all platforms** & I tested again on:
14-
- [ ] Android: Native
14+
- [ ] Android: HybridApp
1515
- [ ] Android: mWeb Chrome
16-
- [ ] iOS: Native
16+
- [ ] iOS: HybridApp
1717
- [ ] iOS: mWeb Safari
1818
- [ ] MacOS: Chrome / Safari
1919
- [ ] MacOS: Desktop
@@ -62,7 +62,7 @@
6262

6363
### Screenshots/Videos
6464
<details>
65-
<summary>Android: Native</summary>
65+
<summary>Android: HybridApp</summary>
6666

6767
<!-- add screenshots or videos here -->
6868

@@ -76,7 +76,7 @@
7676
</details>
7777

7878
<details>
79-
<summary>iOS: Native</summary>
79+
<summary>iOS: HybridApp</summary>
8080

8181
<!-- add screenshots or videos here -->
8282

docs/articles/expensify-classic/bank-accounts-and-payments/bank-accounts/Connect-US-Business-Bank-Account.md

Lines changed: 22 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -39,37 +39,36 @@ You’ll now receive payments for invoices directly into this bank account.
3939
- US-issued ID
4040
- The company address must be in the US
4141
- The controlling officer (the member initiating the connection to the bank account)'s address must be in the US
42-
- The controlling officer must have a US-issued social security number
43-
- **Important:** The person adding the bank account doesn’t need to be an account signer but they must enter their own personal information when connecting the bank account. If another person enters their details, the account will be flagged for review, delaying verification.
42+
- The controlling officer must have a U.S.-issued Social Security number
43+
44+
**Important:** The person adding the bank account doesn’t need to be an account signer but they must enter their own personal information when connecting the bank account. If another person enters their details, the account will be flagged for review, delaying verification.
4445

4546
## Step 1: Connect the Bank Account
4647

47-
1. Hover over **Settings**, then click **Account**.
48-
3. Click the **Wallet** and scroll to **Bank Accounts**.
49-
4. Choose **Add verified bank account** > **Connect to your bank**.
50-
6. Click **Continue**.
51-
7. Enter your phone number and click **Continue**.
52-
8. Select your bank and login.
48+
1. Head to **Settings > Account > Wallet > Bank Accounts**.
49+
2. Choose **Add verified bank account** > **Connect to your bank**.
50+
3. Click **Continue**.
51+
4. Enter your phone number and click **Continue**.
52+
5. Select your bank and login.
5353
- If your bank isn’t listed, click the **X**, select **Connect manually**, enter details, check the box, and click **Save & continue**.
54-
9. Click **Allow**.
55-
10. Ensure the correct account is selected and click **Save & continue**.
56-
11. Enter company details:
54+
6. Click **Allow**.
55+
7. Ensure the correct account is selected and click **Save & continue**.
56+
8. Enter company details:
5757
- **Legal business name**
5858
- **Physical company address** (U.S. address only, no P.O. boxes)
5959
- **Company phone number**
6060
- **Company website** (required for Expensify Card/payments, e.g., `https://www.expensify.com`)
6161
- **Tax Identification Number (TIN)**
6262
- **Company incorporation type, date, and state**
6363
- **Industry Classification Code** ([Find yours here](https://www.sec.gov/corpfin/division-of-corporation-finance-standard-industrial-classification-sic-code-list))
64-
12. Enter your personal information, including a U.S. address and SSN, and click **Save & continue**.
65-
13. Verify your identity by uploading:
64+
9. Enter your personal information, including a U.S. address and SSN, and click **Save & continue**.
65+
10. Verify your identity by uploading:
6666
- A **photo ID** (passport or identity card, must be valid and U.S.-issued)
6767
- A **short video** of yourself
68-
14. If applicable, check **Additional Information**:
68+
11. If applicable, check **Additional Information**:
6969
- If an individual owns 25% or more of the business, their personal details must be provided.
7070
- If the business is a **non-profit**, leave the boxes unchecked.
71-
15. Agree to the terms by selecting the checkboxes.
72-
16. Click **Save & continue**.
71+
12. Agree to the terms by selecting the checkboxes and clicking **Save & continue**.
7372

7473
## Step 2: Verify the Bank Account
7574

@@ -81,10 +80,9 @@ Expensify will send **three test transactions** within **1-2 business days**. To
8180
8281
## Step 3: Add the Bank Account to the Workspace
8382

84-
1. Hover over **Settings**, then click **Workspaces**.
85-
2. Click the desired workspace.
86-
3. Click the **Reimbursement** tab.
87-
4. Click **Direct**, then **Add business bank account** and select the account.
83+
1. Head to **Settings > Workspace > [Workspace Name] > Workflows > Make or track payments**.
84+
2. Click **Direct**.
85+
3. Choose **Add business bank account**, and select the account.
8886

8987
---
9088

@@ -132,7 +130,7 @@ If automatic verification fails, manual verification may require documents like:
132130
- **Business address verification**
133131
- **Bank letter confirming account ownership**
134132

135-
For details, contact Concierge.
133+
For details, contact Concierge Support.
136134

137135
## What if I don’t see all three test transactions?
138136

@@ -168,9 +166,9 @@ Yes, Expensify **cannot process direct payments** for businesses in:
168166
- Casinos & Gambling/Betting Services
169167
- Non-FI, Money Orders
170168
- Wires, Money Orders
171-
- Government Owned Lotteries
172-
- Government Licensed Online Casinos (Online Gambling)
173-
- Government Licensed Horse/Dog Racing
169+
- Government-Owned Lotteries
170+
- Government-Licensed Online Casinos (Online Gambling)
171+
- Government-Licensed Horse/Dog Racing
174172
- Crypto-currency businesses
175173
- Internet gambling
176174
- Marijuana-related businesses

docs/articles/expensify-classic/bank-accounts-and-payments/bank-accounts/Enable-Australian-Reimbursements.md

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,13 @@ If you'd like an option for direct international payments, consider setting up [
1515

1616
# Setting Up ABA Payments
1717

18-
1. Go to **Settings > Workspaces**.
19-
2. Select the workspace you want to configure.
20-
3. Click the **Reports** tab.
21-
4. In the **Report Currency** dropdown, select **AUD A$**.
22-
5. Click the **Reimbursement** tab.
23-
6. Set **Reimbursement Type** to **Indirect**.
24-
7. Enable the toggle.
18+
1. Go to **Settings > Workspaces [Workspace Name] > Overview**.
19+
2. In the **Report Currency** dropdown, select **AUD A$**.
20+
3. Navigate to **Settings > Workspaces [Workspace Name] > Workflows**.
21+
4. Scroll to **Make or track payments**
22+
5. Set **Reimbursement Type** to **Indirect**.
23+
6. Toggle **Collect bank account details** on.
2524

26-
Once enabled, employees can submit expenses in AUD, and you can process reimbursements via ABA files.
25+
Once the setup is complete, employees can submit expenses in AUD, and you can process reimbursements via ABA files.
2726

2827
</div>

0 commit comments

Comments
 (0)