Skip to content

fix: add --broadcast to forge create examples and fix broken links #138

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

Open
erhnysr wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: add --broadcast to forge create examples and fix broken links #138

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f6c7699
fix: add --broadcast to forge create examples and fix broken links
erhnysr Jul 8, 2026
d0d7256
fix: correct broken wallet setup link in agent registration
erhnysr Jul 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion skills/build-on-base/references/agents/register.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ Every agent needs a wallet to sign transactions. Ask the user before doing anyth

1. **Ask: "Do you have a wallet? If yes, share your wallet address."**
2. **If yes** — take the wallet address they provide and move to Phase 2.
3. **If no** — direct them to the Base wallet setup guide: https://docs.base.org/ai-agents/guides/wallet-setup — do not proceed until they have a wallet and can provide their address.
3. **If no** — direct them to create a Base Account at https://base.app — do not proceed until they have a wallet and can provide their address.

---

Expand Down
4 changes: 3 additions & 1 deletion skills/build-on-base/references/deploy-contracts.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ A BaseScan API key is required for the `--verify` flag to auto-verify contracts

> **Agent behavior:** If you have browser access, navigate to the BaseScan site and create the key. Otherwise, ask the user to complete these steps and provide the API key.

1. Go to [basescan.org/myapikey](https://basescan.org/apidashboard) (or [etherscan.io/myapikey](https://etherscan.io/apidashboard) — same account works)
1. Go to [basescan.org/apidashboard](https://basescan.org/apidashboard) (or [etherscan.io/apidashboard](https://etherscan.io/apidashboard) — same account works)
2. Sign in or create a free account
3. Click **Add** to create a new API key
4. Copy the key and set it in your environment:
Expand Down Expand Up @@ -113,6 +113,7 @@ base = { key = "${ETHERSCAN_API_KEY}", url = "https://api.basescan.org/api" }
forge create src/MyContract.sol:MyContract \
--rpc-url https://sepolia.base.org \
--account <keystore-account> \
--broadcast \
--verify \
--etherscan-api-key $ETHERSCAN_API_KEY
```
Expand All @@ -123,6 +124,7 @@ forge create src/MyContract.sol:MyContract \
forge create src/MyContract.sol:MyContract \
--rpc-url https://mainnet.base.org \
--account <keystore-account> \
--broadcast \
--verify \
--etherscan-api-key $ETHERSCAN_API_KEY
```
Expand Down
4 changes: 2 additions & 2 deletions skills/build-on-base/references/migrations/minikit-to-farcaster/mapping.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,7 @@ function AuthButton() {

## useNotification

Notifications require server-side implementation. See [NOTIFICATIONS.md](NOTIFICATIONS.md) for details.
Notifications require server-side implementation. See the [Farcaster Mini App notifications guide](https://miniapps.farcaster.xyz/docs/guides/notifications) for details.

### Before (MiniKit)
```typescript
Expand Down Expand Up @@ -449,4 +449,4 @@ function NotifyButton() {
}
```

See [NOTIFICATIONS.md](NOTIFICATIONS.md) for server-side implementation.
See the [Farcaster Mini App notifications guide](https://miniapps.farcaster.xyz/docs/guides/notifications) for server-side implementation.