Fix @celo/connect README code example

[arthurgousset]

Description

Fixes the code example in the @celo/connect README.

const web3 = new Web3("YOUR_RPC_URL")
const connection = new Connection(web3)

- connection.setProvider()

connection.setProvider() doesn't need to be called explicitly with a CeloProvider, because connection is already configured with a CeloProvider in the constructor:

developer-tooling/packages/sdk/connect/src/connection.ts

Lines 77 to 93 in 0ad9c01

	setProvider(provider: Provider) {
	if (!provider) {
	throw new Error('Must have a valid Provider')
	}
	this._chainID = undefined
	try {
	if (!(provider instanceof CeloProvider)) {
	this.rpcCaller = new HttpRpcCaller(provider)
	provider = new CeloProvider(provider, this)
	}
	this.web3.setProvider(provider as any)
	return true
	} catch (error) {
	console.error(`could not attach provider`, error)
	return false
	}
	}

Related

• CLI and ContractKit support all accepted fee currencies equally, including for transfers and for use as fee currencies #128

Tested

Docs only change

Backwards compatibility

Yes

Documentation

Yes

[changeset-bot]

⚠️ No Changeset found

Latest commit: a884b24

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[arthurgousset]

@nicolasbrugneaux I'm looking for the lines that set the CeloProvider in the connection constructor.
But I can't find it

developer-tooling/packages/sdk/connect/src/connection.ts

Lines 62 to 75 in 0ad9c01

	constructor(readonly web3: Web3, public wallet?: ReadOnlyWallet, handleRevert = true) {
	web3.eth.handleRevert = handleRevert
	this.config = {
	gasInflationFactor: 1.3,
	}
	const existingProvider: Provider = web3.currentProvider as Provider
	this.setProvider(existingProvider)
	// TODO: Add this line with the wallets separation completed
	// this.wallet = _wallet ?? new LocalWallet()
	this.config.from = web3.eth.defaultAccount ?? undefined
	this.paramsPopulator = new TxParamsNormalizer(this)
	}

Can you help me find it again?

[arthurgousset]

Ah my bad, it's here

developer-tooling/packages/sdk/connect/src/connection.ts

Lines 77 to 93 in 0ad9c01

	setProvider(provider: Provider) {
	if (!provider) {
	throw new Error('Must have a valid Provider')
	}
	this._chainID = undefined
	try {
	if (!(provider instanceof CeloProvider)) {
	this.rpcCaller = new HttpRpcCaller(provider)
	provider = new CeloProvider(provider, this)
	}
	this.web3.setProvider(provider as any)
	return true
	} catch (error) {
	console.error(`could not attach provider`, error)
	return false
	}
	}