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

Convert assert in create into a warning #300

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

maurermi
Copy link
Collaborator

Turn the assert statement in create into a warning, this restores the ability to import tokens through Metamask.

Details: When importing tokens, Metamask sends a create call without sender information, triggering an assert failure.

@maurermi
Copy link
Collaborator Author

@HalosGhost This should fix the error we discussed on a call last Friday

Turn the assert statement in create into a warning, this restores the
ability to import tokens through Metamask.

Details: When importing tokens, Metamask sends a create call without
sender information, triggering an assert failure.

Signed-off-by: Michael Maurer <maurer.mi@northeastern.edu>
Copy link
Collaborator

@HalosGhost HalosGhost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaces an assert with a valid/expected code-path to handle MetaMask's behavior changing unexpectedly.

t-ACK, I was able to get all the way through importing tokens into MetaMask without error or crash.

@HalosGhost HalosGhost merged commit cf9060e into mit-dci:trunk Aug 22, 2024
7 checks passed
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