-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: update linea goerli explorer url #7338
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7338 +/- ##
=======================================
Coverage 37.39% 37.39%
=======================================
Files 1052 1052
Lines 28175 28175
Branches 2517 2517
=======================================
Hits 10536 10536
Misses 17040 17040
Partials 599 599 ☔ View full report in Codecov by Sentry. |
## Explanation Update of Linea network Goerli explorer url. ## References See also the [matching patch PR in mobile app](MetaMask/metamask-mobile#7338). ## Changelog ### `@metamask/network-controller` - **CHANGED**: update of Linea network Goerli explorer url. ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate
Linea Goerli explorer URL works as expected for me: goerli_linea.mov |
## Explanation Update of Linea network Goerli explorer url. ## References See also the [matching patch PR in mobile app](MetaMask/metamask-mobile#7338). ## Changelog ### `@metamask/network-controller` - **CHANGED**: update of Linea network Goerli explorer url. ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate
## Explanation Update of Linea network Goerli explorer url. ## References See also the [matching patch PR in mobile app](MetaMask/metamask-mobile#7338). ## Changelog ### `@metamask/network-controller` - **CHANGED**: update of Linea network Goerli explorer url. ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate
Kudos, SonarCloud Quality Gate passed! |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/01b0ad72-050c-40e6-bff4-c2099d01eeb0 |
Kudos, SonarCloud Quality Gate passed! |
Description
Updated Linea Goerli explorer url.
https://explorer.goerli.linea.build
withhttps://goerli.lineascan.build
.@metamask/controller-utils
patchManual testing steps
1. Step1: Send a transaction using the Linea Goerli network
2. Step2: Open the transaction description
3. Step3: Click on the "View on Etherscan" button
4. Step4: Check that it redirects to
https://goerli.lineascan.build
Screenshots/Recordings
If applicable, add screenshots and/or recordings to visualize the before and after of your change.
Before
[screenshot]
After
[screenshot]
Related issues
Patch related to Core PR MetaMask/core#1666
Pre-merge author checklist
Pre-merge reviewer checklist