Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.58k stars 2.92k forks source link

[$250] Web - Sage - Two console errors appear when downloading the Expensify package for Sage Intacct #49261

Closed IuliiaHerets closed 1 month ago

IuliiaHerets commented 2 months ago

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: 9.0.34-3 Reproducible in staging?: Y Reproducible in production?: Y Issue reported by: Applause Internal Team

Action Performed:

  1. Navigate to https://staging.new.expensify.com/
  2. Log in with a new Gmail account
  3. Click on FAB - New workspace
  4. Enable "Accounting" in the "More features" page.
  5. Navigate to "Accounting"
  6. Click on "Sage Intacct" and upgrade the workspace to Control when asked
  7. Click on the "Got it, thanks" button
  8. Click on "Download the Expensify package for Sage Intacct"
  9. Click on "Save" or "Cancel"

Expected Result:

There shouldn't be any console errors.

Actual Result:

Two console errors appear when downloading the Expensify package for Sage Intacct.

Workaround:

Unknown

Platforms:

Screenshots/Videos

1609_1.txt

https://github.com/user-attachments/assets/64c22b4b-f0ce-48d7-9ec4-1f02c08d1825

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021835811720150989795
  • Upwork Job ID: 1835811720150989795
  • Last Price Increase: 2024-09-23
  • Automatic offers:
    • shubham1206agra | Reviewer | 104115002
    • Krishna2323 | Contributor | 104115003
Issue OwnerCurrent Issue Owner: @shubham1206agra
melvin-bot[bot] commented 2 months ago

Triggered auto assignment to @lschurr (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.

IuliiaHerets commented 2 months ago

We think that this bug might be related to #wave-control

melvin-bot[bot] commented 2 months ago

Job added to Upwork: https://www.upwork.com/jobs/~021835811720150989795

melvin-bot[bot] commented 2 months ago

Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External)

Krishna2323 commented 2 months ago

Proposal


Please re-state the problem that we are trying to solve in this issue.

Web - Sage - Two console errors appear when downloading the Expensify package for Sage Intacct.

What is the root cause of that problem?

Due to CORS policy the error is shown when we try to fetch the file using url. https://github.com/Expensify/App/blob/3b850e80143cdf0233866217fcb04afdeec7ef20/src/pages/workspace/accounting/intacct/SageIntacctPrerequisitesPage.tsx#L43

What changes do you think we should make in order to solve the problem?


We should pass shouldOpenExternalLink as true when using fileDownload in SageIntacctPrerequisitesPage.

fileDownload(CONST.EXPENSIFY_PACKAGE_FOR_SAGE_INTACCT, CONST.EXPENSIFY_PACKAGE_FOR_SAGE_INTACCT_FILE_NAME, '', true);

What alternative solutions did you explore? (Optional)

Result

melvin-bot[bot] commented 2 months ago

@lschurr, @shubham1206agra Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

lschurr commented 2 months ago

@shubham1206agra please review the proposal

shubham1206agra commented 2 months ago

@Krishna2323's proposal looks good.

🎀👀🎀 C+ reviewed

melvin-bot[bot] commented 2 months ago

Triggered auto assignment to @stitesExpensify, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

melvin-bot[bot] commented 2 months ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] commented 2 months ago

@lschurr, @stitesExpensify, @shubham1206agra Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] commented 2 months ago

📣 @shubham1206agra 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link Upwork job

melvin-bot[bot] commented 2 months ago

📣 @Krishna2323 🎉 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 📖

Krishna2323 commented 2 months ago

@shubham1206agra, PR ready for review ^

Krishna2323 commented 1 month ago

@lschurr, PR was deployed to production 2 weeks ago, this one is ready for payments.

lschurr commented 1 month ago

@shubham1206agra Can you accept the offer?

lschurr commented 1 month ago

Payment summary:

melvin-bot[bot] commented 1 month ago

@lschurr @stitesExpensify @shubham1206agra @Krishna2323 this issue is now 4 weeks old, please consider:

Thanks!

shubham1206agra commented 1 month ago

@lschurr Offer accepted

melvin-bot[bot] commented 1 month ago

@lschurr, @stitesExpensify, @shubham1206agra, @Krishna2323 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

lschurr commented 1 month ago

This is done!