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

feat: add scroll + fix typos #63

Merged
merged 3 commits into from
Jul 13, 2023
Merged

Conversation

omahs
Copy link
Contributor

@omahs omahs commented Jul 13, 2023

feat: add scroll + fix typos

@joshstevens19
Copy link
Owner

joshstevens19 commented Jul 13, 2023

Will deploy tomorrow thanks for PR

@joshstevens19 joshstevens19 merged commit e4b0e7f into joshstevens19:master Jul 13, 2023
@joshstevens19
Copy link
Owner

Copy link

@Turupawn Turupawn left a comment

Choose a reason for hiding this comment

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

Renaming from Scroll to Scroll Alpha

@@ -66,6 +66,7 @@ The below networks are supported by default, and custom networks can be supporte
| Etherlite | 111 |
| Linea Testnet | 59140 |
| Linea | 59144 |
| Scroll | 534352 |

Choose a reason for hiding this comment

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

Scroll mainnet is not yet launched. This should be renamed to Scroll Alpha

@@ -596,6 +596,7 @@ export class Multicall {
case Networks.kcc:
case Networks.lineaTestnet:
case Networks.linea:
case Networks.scroll:

Choose a reason for hiding this comment

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

Scroll Alpha multicall deployment is at 0x2117f703867a2B7E6813c7e5Edd96bf9a8d8eC30

Choose a reason for hiding this comment

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

And also should be renamed to Networks.scrollAlpha

@@ -50,4 +50,5 @@ export enum Networks {
etherlite = 111,
lineaTestnet = 59140,
linea = 59144,
scroll = 534352,

Choose a reason for hiding this comment

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

This should be renamed to scrollAlpha

@joshstevens19
Copy link
Owner

joshstevens19 commented Jul 18, 2023

@Turupawn please fix with a PR and we will merge and deploy as this has now been merged

PriestBera pushed a commit to PriestBera/ethereum-multicall that referenced this pull request May 18, 2024
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.

None yet

3 participants