Skip to content

Commit

Permalink
Merge branch 'main' into dsilva_fullstoryStartRecordingSettings
Browse files Browse the repository at this point in the history
  • Loading branch information
danieldoglas authored Oct 12, 2024
2 parents 084aaac + e4969b2 commit 99f6ecf
Show file tree
Hide file tree
Showing 217 changed files with 7,746 additions and 1,439 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/buildAndroid.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,14 @@ on:
description: The pull request number associated with this build, if relevant.
type: string
required: false

outputs:
AAB_FILE_NAME:
value: ${{ jobs.build.outputs.AAB_FILE_NAME }}
APK_FILE_NAME:
value: ${{ jobs.build.outputs.APK_FILE_NAME }}
APK_ARTIFACT_NAME:
value: ${{ jobs.build.outputs.APK_ARTIFACT_NAME }}

workflow_dispatch:
inputs:
Expand Down Expand Up @@ -56,6 +59,7 @@ jobs:
outputs:
AAB_FILE_NAME: ${{ steps.build.outputs.AAB_FILE_NAME }}
APK_FILE_NAME: ${{ steps.build.outputs.APK_FILE_NAME }}
APK_ARTIFACT_NAME: ${{ steps.build.outputs.APK_ARTIFACT_NAME }}

steps:
- name: Checkout
Expand Down Expand Up @@ -159,6 +163,7 @@ jobs:
echo "APK_PATH=$apkPath"
echo "APK_FILE_NAME=$(basename "$apkPath")"
echo "SHOULD_UPLOAD_SOURCEMAPS=$SHOULD_UPLOAD_SOURCEMAPS"
echo "APK_ARTIFACT_NAME=${{ inputs.artifact-prefix }}android-artifact-apk" >> "$GITHUB_OUTPUT"
} >> "$GITHUB_OUTPUT"
- name: Upload Android AAB artifact
Expand All @@ -174,7 +179,7 @@ jobs:
continue-on-error: true
uses: actions/upload-artifact@v4
with:
name: ${{ inputs.artifact-prefix }}android-artifact-apk
name: ${{ steps.build.outputs.APK_ARTIFACT_NAME }}
path: ${{ steps.build.outputs.APK_PATH }}

- name: Upload Android sourcemaps artifact
Expand Down
17 changes: 0 additions & 17 deletions .github/workflows/deployBlocker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,23 +29,6 @@ jobs:
escaped_title=$(echo "$GH_ISSUE_TITLE" | sed -e 's/&/\&amp;/g; s/</\&lt;/g; s/>/\&gt;/g; s/"/\&quot;/g; s/'"'"'/\&#039;/g; s/|/\&verbar;/g')
echo "GH_ISSUE_TITLE=$escaped_title" >> "$GITHUB_ENV"
- name: 'Post the issue in the #expensify-open-source slack room'
if: ${{ success() }}
uses: 8398a7/action-slack@v3
with:
status: custom
custom_payload: |
{
channel: '#expensify-open-source',
attachments: [{
color: "#DB4545",
text: '💥 New Deploy Blocker: <${{ github.event.issue.html_url }}|${{ env.GH_ISSUE_TITLE }}>. If you have any idea which PR could be causing this, please comment in the issue.'
}]
}
env:
GITHUB_TOKEN: ${{ github.token }}
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}

- name: Comment on deploy blocker
run: |
gh issue comment ${{ github.event.issue.number }} --body "$(cat <<'EOF'
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/e2ePerformanceTests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ jobs:
uses: actions/download-artifact@v4
id: downloadBaselineAPK
with:
name: baseline-${{ needs.prep.outputs.BASELINE_VERSION }}-android-artifact-apk
name: ${{ needs.buildBaseline.outputs.APK_ARTIFACT_NAME }}
path: zip
# Set github-token only if the baseline was built in this workflow run:
github-token: ${{ needs.prep.outputs.BASELINE_ARTIFACT_WORKFLOW_ID && github.token }}
Expand All @@ -139,7 +139,7 @@ jobs:
uses: actions/download-artifact@v4
id: downloadDeltaAPK
with:
name: delta-${{ needs.prep.outputs.DELTA_REF }}-android-artifact-apk
name: ${{ needs.buildDelta.outputs.APK_ARTIFACT_NAME }}
path: zip

- name: Rename delta APK
Expand Down
6 changes: 3 additions & 3 deletions android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ apply from: project(':react-native-config').projectDir.getPath() + "/dotenv.grad
/* Fullstory settings */
fullstory {
org 'o-1WN56P-na1'
enabledVariants 'all'
enabledVariants 'production'
logcatLevel 'debug'
recordOnStart false
}
Expand Down Expand Up @@ -110,8 +110,8 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
multiDexEnabled rootProject.ext.multiDexEnabled
versionCode 1009004604
versionName "9.0.46-4"
versionCode 1009004800
versionName "9.0.48-0"
// Supported language variants must be declared here to avoid from being removed during the compilation.
// This also helps us to not include unnecessary language variants in the APK.
resConfigs "en", "es"
Expand Down
37 changes: 0 additions & 37 deletions assets/images/companyCards/card-bank_of_america.svg

This file was deleted.

35 changes: 0 additions & 35 deletions assets/images/companyCards/card-other.svg

This file was deleted.

30 changes: 15 additions & 15 deletions desktop/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion desktop/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"dependencies": {
"electron-context-menu": "^2.3.0",
"electron-log": "^4.4.8",
"electron-updater": "^6.3.5",
"electron-updater": "^6.3.6",
"mime-types": "^2.1.35",
"node-machine-id": "^1.1.12"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ This error occurs when the account applied as a category to the expense in Expen
5. Click on the pencil icon on the right to check if you have "In multiple accounts" selected:
6. If "In multiple accounts" is selected, go to Chart of Accounts and click Edit for the account in question.
7. Check the billable option and select an income account within your chart of accounts
8. Sync your QuickBooks Online connection in Settings > Workspaces > [click workspace] > Connections.
8. Sync your QuickBooks Online connection in Settings > Workspaces > [workspace name] > Connections.
9. Open the report and click the Export to button and then the QuickBooks Online option.

# ExpensiError QBO046: Feature Not Included in Subscription
Expand All @@ -40,11 +40,11 @@ _Please note: Self Employed is not supported:_

**Why does this happen?**

QuickBooks Online requires all expenses exported from Expensify to use a category matching an account in your chart of accounts. If a category from another source is used, QuickBooks Online will reject the expense. This errors occurs when an expense on the report has a category applied that is not valid in QuickBooks Online.
QuickBooks Online requires all expenses exported from Expensify to use a category matching an account in your chart of accounts. If a category from another source is used, QuickBooks Online will reject the expense. This error occurs when an expense on the report has a category applied that is not valid in QuickBooks Online.

## How to fix it

1. Sync your QuickBooks Online connection in Expensify from Settings > Workspaces > [click workspace] > Connections, and click the **Sync Now** button.
1. Sync your QuickBooks Online connection in Expensify from Settings > Workspaces > [workspace name] > Connections, and click the **Sync Now** button.
2. Review the expenses on the report. If any appear with a red _Category no longer valid_ violation, recategorize the expense until all expenses are violation-free.
3. Click the **Export t**o button and then the **QuickBooks Online** option.
- If you receive the same error, continue.
Expand All @@ -56,7 +56,7 @@ QuickBooks Online requires all expenses exported from Expensify to use a categor

**Why does this happen?**

This error occurs when you have an Employee Record set up with the employee's name, which prevents the Expensify integration from automatically creating the Vendor Record with the same name, since QuickBooks Online won't allow you to have an employee and vendor with the same name.
This error occurs when you have an Employee Record set up with the employee's name. This prevents the Expensify integration from automatically creating the Vendor Record with the same name since QuickBooks Online won't allow you to have an employee and vendor with the same name.

## How to fix it

Expand All @@ -67,7 +67,7 @@ There are two different ways you can resolve this error.
1. Log into QuickBooks Online.
2. Access the Employee Records for your submitters.
3. Edit the name to differentiate them from the name they have on their account in Expensify.
4. Sync your QuickBooks Online connection in Settings > Workspaces > [click workspace] > Connections.
4. Sync your QuickBooks Online connection in Settings > Workspaces > [workspace name] > Connections.
5. Open the report and click the Export to button and then the QuickBooks Online option.

**Option 2**:
Expand All @@ -85,7 +85,7 @@ This error occurs when you are exporting reimbursable expenses as Journal Entrie

There are three different ways you can resolve this error.
- Select a different type of export for reimbursable expenses under Settings > Workspaces > [worksapce name] > Connections > Configure > Export tab.
- Enable _Automatically Create Entities_ under Settings > Workspaces > [worksapce name] > Connections > Configure > Advanced to create vendor records automatically.
- Enable _Automatically Create Entities_ under Settings > Workspaces > [workspace name] > Connections > Configure > Advanced to create vendor records automatically.
- Manually create vendor records in QuickBooks Online for each employee.

# ExpensiError QBO099: Items marked as billable must have sales information checked
Expand All @@ -97,7 +97,7 @@ This error occurs when an Item category on an expense does not have sales inform
## How to fix it

1. Log into QuickBooks Online.
2. Navigate to to your items list.
2. Navigate to your items list.
3. Click **Edit** to the right of the item used on the report with the error. Here you will see an option to check either "Sales" or "Purchasing".
4. Check the option for **Sales**.
5. Select an income account.
Expand Down Expand Up @@ -146,12 +146,12 @@ This error occurs because the currency on the Vendor record in QuickBooks Online

1. Log into QuickBooks Online.
2. Open the vendor record.
3. Update the record to use with the correct A/P account, currency and an email matching their Expensify email.
You can find the correct Vendor record by exporting your QuickBooks Online [vendor list](https://community.expensify.com/home/leaving?allowTrusted=1&target=https%3A%2F%2Fqbo.intuit.com%2Fapp%2Fvendors) to a spreadsheet (click the export icon on the right-hand side of the page), and search for the email address of the person who submitted the report.
3. Update the record to use with the correct A/P account, currency, and email matching their Expensify email.
You can find the correct Vendor record by exporting your QuickBooks Online vendor list to a spreadsheet (click the export icon on the right-hand side of the page), and search for the email address of the person who submitted the report.

If you have multiple Vendors with different currencies with the same email, Expensify is likely trying to export to the wrong one.

1. Try removing the email address from the vendor in QuickBooks Online you aren't trying to export to.
1. Try removing the email address from the vendor in QuickBooks Online that you aren't trying to export to.
2. Sync your QuickBooks Online connection in Settings > Workspaces > [workspace name] > Connections.
3. Open the report and click the **Export to** button and then the **QuickBooks Online** option.

Expand All @@ -160,21 +160,21 @@ If this still fails, you'll need to confirm that the A/P account selected in Exp
1. Navigate to Settings > Workspaces > [workspace name] > Connections.
2. Under the Exports tab check that both A/P accounts are the correct currency.

# Why are company card expenses exporting to the wrong account in QuickBooks Online?
# Why are company card expenses exported to the wrong account in QuickBooks Online?

Multiple factors could be causing your company card transactions to export to the wrong place in your accounting system, but the best place to start is always the same.

1. First, confirm that the company cards have been mapped to the correct accounts in Settings > Domains > Company Cards > click the **Edit Export button** for the card to view the account.
2. Next, confirm the expenses in question have been imported from the company card?
- Only expenses that have the Card+Lock icon next to them will export according to the mapping settings that you configure in the domain settings.
2. Next, confirm the expenses in question have been imported from the company card.
- Only expenses with the Card+Lock icon next to them will export according to the mapping settings that you configure in the domain settings.

It’s important to note that expenses imported from a card linked at the individual account level, expenses created from a SmartScanned receipt, and manually created cash expenses will export to the default bank account selected in your connection's configuration settings.

**Is the report exporter a domain admin?**

The user exporting the report must be a domain admin. You can check the history and comment section at the bottom of the report to see who exported the report.

If your reports are being exported automatically by Concierge, the user listed as the Preferred Exporter under Settings > Workspaces > [workspaces name] > Connections > click **Configure** must be a domain admin as well.
If your reports are being exported automatically by Concierge, the user listed as the Preferred Exporter under Settings > Workspaces > [workspace name] > Connections > click **Configure** must also be a domain admin.

If the report exporter is not a domain admin, all company card expenses will export to the bank account set in Settings > Workspaces > [workspace name] > Connections > click **Configure** for non-reimbursable expenses.

Expand Down
Loading

0 comments on commit 99f6ecf

Please sign in to comment.