docs: tw589-seq-timing-adjustments#3024
Merged
pete-vielhaber merged 22 commits intomasterfrom Feb 11, 2026
Merged
Conversation
Contributor
pete-vielhaber
commented
Jan 20, 2026
pete-vielhaber
commented
- Generated content for sequencer timing adjustments
- needs thorough review
- questioning placement; this may be better combined into a single page and placed under Arbitrum Node Runners (under Launch Arbitrum Chain)
This reverts commit a60b607.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
EmreDincoglu
requested changes
Jan 28, 2026
Contributor
EmreDincoglu
left a comment
EmreDincoglu
left a comment
There was a problem hiding this comment.
Im decently knowledgeable about most of these topics but if anyone else could review it I think that would be good.
| ```typescript | ||
| import { prepareNodeConfig } from '@arbitrum/orbit-sdk'; | ||
|
|
||
| const nodeConfig = await prepareNodeConfig({ |
Contributor
There was a problem hiding this comment.
So im looking at the chain SDK and while you should override the node config file. I do not see any parameters you can add to prepareNodeConfig function (Specifically for overriding), and looking where the execution part is, it seems to be hard coded, so needing to override the info is correct, but it cannot be done via using the prepareNodeConfig function, it can only be done after you have received the var containing the nodeConfig information
…nfig-sequencer-timing-adjustments.mdx Co-authored-by: Emre Dincoglu <85258206+EmreDincoglu@users.noreply.github.com>
…nfig-sequencer-timing-adjustments.mdx Co-authored-by: Emre Dincoglu <85258206+EmreDincoglu@users.noreply.github.com>
pete-vielhaber
commented
Jan 29, 2026
…nfig-sequencer-timing-adjustments.mdx
pete-vielhaber
commented
Jan 29, 2026
EmreDincoglu
requested changes
Feb 5, 2026
| @@ -35,16 +35,13 @@ import { prepareNodeConfig } from '@arbitrum/orbit-sdk'; | |||
| const nodeConfig = await prepareNodeConfig({ | |||
Contributor
There was a problem hiding this comment.
I would move the if statements out of the function call since it looks like you send them into the function. Other than that as long as in this tutorial the goal is to set some of these values aka the max-block-speed and max-delay then this should be good
EmreDincoglu
approved these changes
Feb 5, 2026
Jason-W123
reviewed
Feb 6, 2026
Jason-W123
reviewed
Feb 6, 2026
Jason-W123
reviewed
Feb 9, 2026
Jason-W123
reviewed
Feb 9, 2026
Jason-W123
reviewed
Feb 9, 2026
pete-vielhaber
commented
Feb 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.