[FIX] Patch assets-controller to not use BigInt #6305
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Development & PR Process
release-xx
label to identify the PR slated for a upcoming release (will be used in release discussion)needs-dev-review
label when work is completedneeds-qa
label when dev review is completedQA Passed
label when QA has signed offDescription
This PR patches @metamask/assets-controller to comment out BigInt related code for now. We will reintroduce it once RN 0.71.6 upgrade is finished.
Screenshots/Recordings
bigintfix.mov
Issue
Description
Whenever I try to make an Approve tx, the app crashes with this errror
Can't find variable: BigInt
[only-android]
After first reporting it here, we've seen that it's a separate issue, happening on the RC branch
6.4
Screenshots
bigint-module-not-found.mp4
Steps to reproduce the behavior
Can't find variable: BigInt
Issue: #6307
Progresses #???
Checklist