-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Jl/mmp 869/eip 6963 (tmashuang edit) #263
Conversation
Definitely messed something up, unable to run e2e tests with these changes. |
I probably shouldn't be the one to ultimately review this since i originally made these changes |
…error message of same name, rdns, and image of existing provider uuid.
8faa2de
to
f79e00b
Compare
I am marking this ready for review. Idk what is going on the extension side of the e2e tests, hopefully this update won't break them. My local issues with them consist of not being able to run the entire suite to check for issues against this PR and inconsistent accounts/fixture data/connecting issues. I didnt find any issues manual locally testing with a dev build and this pr. |
This is alternative to #243 where it has been rebased with main since its creation date. Also adds eip6963 provider detection on page load.
eip6963-test-dapp.mov