Skip to content

fix/standardize onchain spelling #1071

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.

Sign up for GitHub

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

Jump to bottom
Open
Earnwithalee7890 wants to merge 18 commits into
base: master
Choose a base branch
from
Open

fix/standardize onchain spelling #1071

Earnwithalee7890 wants to merge 18 commits into from

Conversation

@Earnwithalee7890
Copy link

@Earnwithalee7890 Earnwithalee7890 commented Jan 30, 2026

What changed? Why?
Standardized "on-chain" to "onchain" across multiple documentation files to align with Base branding guidelines.
Base uses "onchain" (one word, lowercase 'c') as the preferred spelling.

Notes to reviewers

  • Changes applied to base-services-hub.mdx, bridge-tokens-with-layerzero.mdx, data-indexers.mdx, and base-app-coins.mdx.
  • Checked content to ensure readability and flow were maintained.
  • Code references like Ether.onChain were preserved.

How has it been tested?

  • grep search for "on-chain" to identify occurrences.
  • Manual verification to distinguish between text usage (updated) and code usage (preserved).

Earnwithalee7890 and others added 18 commits January 24, 2026 02:36
@Earnwithalee7890
docs: restructure testnets to separate active from deprecated networks
e9b5bf0
@Earnwithalee7890
Update iframe-theme.js to improve documentation coverage
ddf50ce
@Earnwithalee7890
Update filtering-an-array-sbs.mdx to improve documentation coverage
de7bb89
@Earnwithalee7890
Update account-abstraction-on-base-using-privy-and-the-base-paymaster…
c241fd4 
….mdx to improve documentation coverage
@Earnwithalee7890
Update introduction-to-providers.mdx to improve documentation coverage
71d0ef7
@Earnwithalee7890
Update useReadContract.mdx to improve documentation coverage
917729e
@Earnwithalee7890
Update useSimulateContract.mdx to improve documentation coverage
78281ae
@Earnwithalee7890
Update erc-721-sbs.mdx to improve documentation coverage
e132a3d
@Earnwithalee7890
Update complex-onchain-nfts.mdx to improve documentation coverage
a5aaceb
@Earnwithalee7890
Update signature-mint.mdx to improve documentation coverage
cb6ed56
@Earnwithalee7890
Update simple-onchain-nfts.mdx to improve documentation coverage
5257f5a
@Earnwithalee7890
Update thirdweb-unreal-nft-items.mdx to improve documentation coverage
c6c1efa
@Earnwithalee7890
Update overview.mdx to improve documentation coverage
cf0f8ff
@Earnwithalee7890
Update quality-bar.mdx to improve documentation coverage
4f4a036
@Earnwithalee7890
Update submission-guidelines.mdx to improve documentation coverage
6f87064
@Earnwithalee7890
Update error-handling.mdx to improve documentation coverage
3c1d78c
@Earnwithalee7890
Merge branch 'base:master' into master
d8ce5ec
@Earnwithalee7890
style(docs): standardize onchain spelling
4d34ee9
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 30, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Earnwithalee7890 @cb-heimdall