Skip to content

Conversation

@tabaktoni
Copy link
Member

@tabaktoni tabaktoni commented Nov 6, 2023

Motivation and Resolution

fix #803

  • Set default provider to RPC Provider
  • Update RPC Provider with option 'empty nodeUrl' - will use random public goerli nodeUrl from the known nodes
  • Update RPC Provider with option 'NetworkName for nodeUrl' - provide NetworkName and random public nodeUrl will be used for specific network
  • Updated test setup for default RPC Provider and new options

RPC version (if applicable)

0.5

Usage related changes

mentioned

Checklist:

  • Performed a self-review of the code
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Linked the issues which this PR resolves
  • Documented the changes in code (API docs will be generated automatically)
  • Updated the tests
  • All tests are passing

@tabaktoni tabaktoni linked an issue Nov 6, 2023 that may be closed by this pull request
Copy link
Collaborator

@penovicp penovicp left a comment

Choose a reason for hiding this comment

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

Guessing there are more test changes planned for this PR? Or will that come in a separate one?

Copy link
Collaborator

@penovicp penovicp left a comment

Choose a reason for hiding this comment

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

One small side note, lgtm

? new RpcProvider({ nodeUrl: process.env.TEST_RPC_URL })
: new SequencerProvider({ baseUrl: process.env.TEST_PROVIDER_BASE_URL || '' });

const provider = new RpcProvider({ nodeUrl: process.env.TEST_RPC_URL });
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might be worth considering moving this change to a future PR so the Sequencer tests can pass (one last time
🤞).

Copy link
Member Author

Choose a reason for hiding this comment

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

True

Copy link
Member Author

Choose a reason for hiding this comment

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

Even doe we can do additional fixes and tests on the final v5 when devnet release

@tabaktoni tabaktoni changed the base branch from feat/rpc-v5 to next-version November 9, 2023 10:23
@tabaktoni tabaktoni changed the title Default RPC Provider Deprecations & Default RPC Provider Nov 9, 2023
@tabaktoni tabaktoni merged commit 555335a into next-version Nov 9, 2023
@tabaktoni tabaktoni mentioned this pull request Nov 9, 2023
6 tasks
@github-actions
Copy link
Contributor

🎉 This PR is included in version 5.23.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@penovicp penovicp deleted the feat/default-rpc-provider branch December 6, 2023 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Default RPC Provider

4 participants