Add back the QR code download button

[lakchote]

QR code download doesn't work anymore (as of today) since the new arch (Fabric PR) got merged.

TL;DR: react-native-view-shot does not support new arch yet.

It was decided here to hide the Download button for QR code in the meantime, to prevent users from encountering bugs and degrading user experience.

When the update 0.74 for react-native is available, we should:

• use it so react-native-view-shot works again as intended
• Revert PR here and show back the Download button.

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01d1641a383b71483e
• Upwork Job ID: 1778781286743756800
• Last Price Increase: 2024-04-12

[ShridharGoel]

Proposal

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

QR code download using react-native-view-shot doesn't work after moving to the new arch.

What is the root cause of that problem?

The react-native-view-shot library has introduced the new architecture support in latest version, but we are still using 3.8.0. (gre/react-native-view-shot#516)

Also, these changes in react-native need to be included.

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

Add patch for the view-shot library with the changes for the new architecture or bump the version to 4.0.0-alpha.0.
Add patch for these changes in react-native.

Note: This is same as my proposal here.

[lakchote]

@ShridharGoel thanks for your proposal but please refrain posting proposals in issues where the Help wanted label is not present.

Instead, post (like you did) in the issue where proposals are allowed, thank you!

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @ahmedGaber93 (Internal)

[lakchote]

Still waiting for RN 0.74 to be effective. Tracking PR here.

[lakchote]

Still waiting for RN 0.74 to be effective. Tracking PR here.

Still waiting for RN 0.74 PR to be available. Putting this as a monthly in the meantime.

[ahmedGaber93]

Not overdue, Still hold on #37374

[lakchote]

Not overdue, Still hold on #37374

Same.

[ahmedGaber93]

Not overdue, Still hold on #37374

[ahmedGaber93]

I will unassign me to clean up my k2.

[s77rt]

Alternative solution: @s77rt/react-native-viewshot

[lakchote]

PR #37374 was merged, will take a look to add it back as soon as I can.

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[lakchote]

PR didn't solve the issue, see #49868 (comment). We're going to wait for a new stable version.

[brunovjk]

I don't have a physical iOS device, I only have Android. I'm available to test if necessary. Thanks.

[lakchote]

Still waiting for a stable version of react-native-viewshot v4.0.0 (see #49868 (comment)).

[lakchote]

Still waiting for a stable version of react-native-viewshot v4.0.0 (see #49868 (comment)).

Stable version is now available. As such, I've raised a new PR here.

[fedirjh]

@lakchote Can you please assign me as C+ reviewer in #53757 ? Thank you.