Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: TypeError: can't convert BigInt to number #499

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

cromatikap
Copy link

After the following

$ npx create-react-app my-safe-app --template @safe-global/cra-template-safe-app
$ cd my-safe-app
$ npm i
$ npm run build

The build would display a blank page with TypeError: can't convert BigInt to number error in console.

I applied this solution and it fixed the issue.

…er is served after having done `npm run build`
@changeset-bot
Copy link

changeset-bot bot commented Aug 7, 2023

🦋 Changeset detected

Latest commit: 3d03006

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@safe-global/cra-template-safe-app Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@cromatikap
Copy link
Author

the CLA link is broken 😅
Screenshot 2023-08-07 at 13 11 12

@cromatikap
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@cromatikap
Copy link
Author

recheck

@mmv08 mmv08 changed the base branch from main to development August 8, 2023 12:44
@mmv08
Copy link
Member

mmv08 commented Aug 8, 2023

The CLA should be available here: https://safe.global/cla

Could you please also include a changeset? Thank you. The comment from the changeset bot should guide you through the process of adding one

@cromatikap
Copy link
Author

Added the changeset 👍 , anything else?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants