-
Notifications
You must be signed in to change notification settings - Fork 3
[wip] feat: adds mainnet chain to constants crate #138
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
base: main
Are you sure you want to change the base?
Conversation
47790e3 to
074fc32
Compare
| pub const MAINNET: SignetConstants = SignetConstants::new(MAINNET_SYS, MAINNET_ENV); | ||
|
|
||
| /// The URL of the Transaction Cache endpoint (empty for mainnet placeholder). | ||
| pub const TX_CACHE_URL: &str = "TODO"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only thing missing is the tx cache url cc @rswanson
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Evalir let's get this updated and moved to review asap 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was already updated? As long as we're good with the deploy height being unfinalized I'm ok with marking this as rfr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, didn't realize it had been pushed up. Will get you the finalized deploy height number and we can move this to review 👍
|
deploy height is potentially not finalized |
074fc32 to
fc02bfb
Compare

[wip] feat: adds mainnet chain to constants crate
This PR adds mainnet values to the constants crate in preparation for mainnet deployment.
Closes ENG-1339