Open jamesdeanexpensify opened 1 week ago
Triggered auto assignment to @slafortune (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
Job added to Upwork: https://www.upwork.com/jobs/~010acf66c02b3a1adb
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ikevin127 (External
)
Triggered auto assignment to @dannymcclain (Design
), see these Stack Overflow questions for more details.
When Credit card is selected, we currently show Account When Debit card is selected, we currently show Account
Show in both menu title and header title.
This is a copy update feature.
Remember to include the vendor_bill
>> accountsPayable
check as well.
switch (nonReimbursableExpensesExportDestination) {
case CONST.QUICKBOOKS_NON_REIMBURSABLE_EXPORT_ACCOUNT_TYPE.DEBIT_CARD:
return translate('workspace.qbo.bankAccount');
case CONST.QUICKBOOKS_NON_REIMBURSABLE_EXPORT_ACCOUNT_TYPE.CREDIT_CARD:
return translate('workspace.qbo.creditAccount');
case CONST.QUICKBOOKS_NON_REIMBURSABLE_EXPORT_ACCOUNT_TYPE.VENDOR_BILL:
return translate('workspace.qbo.accountsPayable');
default:
return undefined;
}
@gijoe0295's proposal looks good to me. Root cause is not needed for this issue since it's a copy update Improvement
. The proposed solution is enough to move forward with assignment, the below mentions can be handled during PR:
creditAccount
translation variable for both english and spanish, being Credit card account
/ Cuenta tarjeta de crédito
- make sure to validate the translations (especially for spanish) ⚠️.switch
default case should return translate('workspace.qbo.account')
instead of undefined
.🎀👀🎀 C+ reviewed
@Beamanator Any idea where should we keep the reusable function mentioned at (3.) since it's used in 2 components ? I did not notice any QBOUtils file or something similar.
Triggered auto assignment to @Beamanator, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
📣 @ikevin127 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!
📣 @gijoe0295 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻 Keep in mind: Code of Conduct | Contributing 📖
@gijoe0295 Any plans to open a PR for this soon ? If not, please let us know so we can re-assign another Contributor and move this forward.
PR will be opened in the next hour.
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: v9.0.1-9 (I'm using Chrome web app staging) Reproducible in staging?: Yes Reproducible in production?: Yes If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): james@expensify.com Logs: N/A Expensify/Expensify Issue URL: N/A Issue reported by: james@expensify.com Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1718987997426219?thread_ts=1718837013.444269&cid=C036QM0SLJK
Action Performed:
Export
>Export company cards as
Credit card
andDebit card
to see the screen that results (See video below with voiced walkthrough)Expected Result:
Credit card
is selected, we should showCredit card account
Debit card
is selected, we should showBank account
(See video below with voiced walkthrough)Actual Result:
Credit card
is selected, we currently showAccount
Debit card
is selected, we currently showAccount
(See video below with voiced walkthrough)Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
https://github.com/Expensify/App/assets/22160301/f29a6c2f-3573-4cb8-9570-bf8122b80428
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @ikevin127