[PAY] 2 copies of react-pdf are included in the web js bundle

[janicduplessis]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: Latest
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):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @janicduplessis
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1714407724836209?thread_ts=1714247539.428689&cid=C05LX9D6E07

Action Performed:

• Inspect js bundle using the analyze-packages script
• See that there are 2 copies of react-pdf

Expected Result:

There should only be 1 copy of react-pdf

Actual Result:

There are 2 copies of react-pdf

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

N/A

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

[melvin-bot]

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

[melvin-bot]

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

[mallenexpensify]

Assigned to @ahmedGaber93 as they're reviewing the PR.

[roryabraham]

Reviewed and merged the PR after being tagged in slack

[melvin-bot]

This issue has not been updated in over 15 days. @mallenexpensify, @ahmedGaber93, @roryabraham eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[janicduplessis]

This has been merged already, can we close it?

[mallenexpensify]

Thanks for the ping @janicduplessis . @ahmedGaber93 , you're due $250 here, right? Assuming so, can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~012eb7ba95c3bdb089
I

[ahmedGaber93]

Yes, right. Offer accepted thanks @mallenexpensify

[mallenexpensify]

Contributor+: @ahmedGaber93 paid $250 via Upwork.

Thanks!