Skip to content

docs(onchain-clients): remove empty "Fetching the Current Block Number" section - #668

Open
callumweb3 wants to merge 1 commit into
inkonchain:mainfrom
callumweb3:patch-4
Open

callumweb3 wants to merge 1 commit into
inkonchain:mainfrom
callumweb3:patch-4

Conversation

@callumweb3

Copy link
Copy Markdown

What

onchain-clients.mdx promises a code snippet but has none - between the intro line and ## Viem there is no code block.

Why remove

The same call is already documented in step 3 of both client sections:

  • Ethers.js: await provider.getBlockNumber()
  • Viem: await client.getBlockNumber()

So the section is a duplicate of step 3 and adds nothing new.

Change

Remove the empty heading + promise line. No info is lost.

…r" section

## What

`onchain-clients.mdx` (lines ~38–41) promises a code snippet but has none —
between the intro line and `## Viem` there is no code block.

## Why remove

The same call is already documented in step 3 of both client sections:

- Ethers.js: `await provider.getBlockNumber()`
- Viem: `await client.getBlockNumber()`

So the section is a duplicate of step 3 and adds nothing new.

## Change

Remove the empty heading + promise line. No info is lost.

## Alternative

If a standalone example is preferred, see PR #<номер_первого_PR>, which fills
the section. Only one should be merged.
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.

1 participant