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: fixed metamask commands to support v11.15.5 #1132

Merged
merged 11 commits into from
May 28, 2024

Conversation

fullstackninja864
Copy link
Contributor

@fullstackninja864 fullstackninja864 commented May 20, 2024

Motivation and context

Current version of synpress was missing some commands to setup latest metamask v11.15.5.
In this PR we have fixed First time flow, Main page, Notification and setting page selectors.

Does it fix any issue?

#(issue)

Other useful info

N/A

Quality checklist

  • I have performed a self-review of my code.

⚠️👆 Delete any section you see irrelevant before submitting the pull request 👆⚠️

Copy link

vercel bot commented May 20, 2024

@fullstackninja864 is attempting to deploy a commit to the Synpress Team on Vercel.

A member of the Team first needs to authorize it.

drptbl and others added 3 commits May 26, 2024 13:07
Signed-off-by: drptbl <jakub.mucha@icloud.com>
Signed-off-by: drptbl <jakub.mucha@icloud.com>
@fullstackninja864
Copy link
Contributor Author

@drptbl In my latest commit i have added packageManager (as testcase were failing because of it). let me know if there is any change required to get this issue fixed .
Thanks

@abdullah-almesbahi
Copy link

@drptbl we need this PR to be merged ASAP, e2e stopped working because the new version of google chrome 125 is not compatible with metamask 10.25.0

@drptbl
Copy link
Collaborator

drptbl commented May 27, 2024

@drptbl we need this PR to be merged ASAP, e2e stopped working because the new version of google chrome 125 is not compatible with metamask 10.25.0

This pull request is not working. Tests are failing.

@fullstackninja864
Copy link
Contributor Author

Fixing it, Thankyou for pointing out

@fullstackninja864
Copy link
Contributor Author

@drptbl Can you review the changes

r3kt-eth added 3 commits May 28, 2024 16:12
Signed-off-by: r3kt.eth <so@r3kt.io>
Signed-off-by: r3kt.eth <so@r3kt.io>
Signed-off-by: r3kt.eth <so@r3kt.io>
@drptbl drptbl merged commit 45d1d6d into Synthetixio:dev May 28, 2024
2 of 10 checks passed
drptbl added a commit that referenced this pull request May 28, 2024
* Remove leftover TODOs

* Remove text based locators

* Add `Known problems with MetaMask` section

* Remove Promise wrap from `cy.setupMetamask()` (#927)

* Fix localized Chrome's extension id (#928)

* Fix localized Chrome's extension id

* Improve id handling

---------

Co-authored-by: Piotr Frankowski <piotr@f-dev.io>

* Lint

* Feature/revoke permission to all (#932)

* Fix typo in Permission word

* Add permission revoking actions

* Add tests for permission revoking actions

* Regenerate synpress commands file

* Add `switchNetwork` option to `acceptAccess` function

* Add new release section to README

* Use `goerli` for testing (#1082)

* Use `goerli` for testing

* Trigger tests

* Add `shouldWaitForPopupClosure` option to approvals and txs (#1081)

* fix: fixed metamask commands to support v11.15.5 (#1132)

* release: v3.7.2-beta.9

Signed-off-by: Jakub <ci@drptbl.dev>

* feature: updated lib to support metamask 11.15.5 version

* use latest pnpm

Signed-off-by: drptbl <jakub.mucha@icloud.com>

* update metamask version

Signed-off-by: drptbl <jakub.mucha@icloud.com>

* added packageManager in package.json

* textcase fix

* click on proceedAnyway

Signed-off-by: r3kt.eth <so@r3kt.io>

* add pnpm

Signed-off-by: r3kt.eth <so@r3kt.io>

* version update

Signed-off-by: r3kt.eth <so@r3kt.io>

---------

Signed-off-by: Jakub <ci@drptbl.dev>
Signed-off-by: drptbl <jakub.mucha@icloud.com>
Signed-off-by: r3kt.eth <so@r3kt.io>
Co-authored-by: Jakub <ci@drptbl.dev>
Co-authored-by: Harsh Rathi <harsh@FV3P2P7KXV.local>
Co-authored-by: drptbl <jakub.mucha@icloud.com>
Co-authored-by: r3kt.eth <so@r3kt.io>

---------

Signed-off-by: Jakub <ci@drptbl.dev>
Signed-off-by: drptbl <jakub.mucha@icloud.com>
Signed-off-by: r3kt.eth <so@r3kt.io>
Co-authored-by: duckception <spidiritsu@gmail.com>
Co-authored-by: Peter F <pietras93@gmail.com>
Co-authored-by: Piotr Frankowski <piotr@f-dev.io>
Co-authored-by: Rafał Majchrzak <rafal2228@gmail.com>
Co-authored-by: Harsh R <53080940+fullstackninja864@users.noreply.github.com>
Co-authored-by: Harsh Rathi <harsh@FV3P2P7KXV.local>
Co-authored-by: r3kt.eth <so@r3kt.io>
@drptbl
Copy link
Collaborator

drptbl commented May 28, 2024

Hey @fullstackninja864, thanks for your awesome work here. There are still small things to fix, but it's working.

Please get in touch with me on Telegram for a small bounty for your work (@drptbl).

Thanks,
Jakub.

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.

4 participants