Skip to content

Conversation

@bradleycamacho
Copy link
Contributor

@bradleycamacho bradleycamacho commented Oct 21, 2025

Contains quickstart doc for Actions

@bradleycamacho bradleycamacho requested review from a team and 0xmariniere as code owners October 21, 2025 21:01
@bradleycamacho bradleycamacho changed the base branch from main to mintlify October 21, 2025 21:01
@netlify
Copy link

netlify bot commented Oct 21, 2025

Deploy Preview for docs-optimism failed. Why did it fail? →

Name Link
🔨 Latest commit f23b1e3
🔍 Latest deploy log https://app.netlify.com/projects/docs-optimism/deploys/68f7f498b974600008020d7f

@its-everdred its-everdred changed the title Actions draft content Actions Quickstart content Nov 5, 2025
@its-everdred its-everdred marked this pull request as ready for review November 5, 2025 19:57
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can omit this

Copy link
Contributor

Choose a reason for hiding this comment

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

good catch, thank you

@@ -0,0 +1,536 @@
---
title: Integrating DeFi with Actions SDK
description: Lend, Borrow, Swap, Pay in your app with a single dependency.
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: is it a single dependency if you also need the wallet provider? What if we use the description that's on your landing page? "Perform DeFi actions with lightweight, composable, and type-safe modules."

Copy link
Contributor

Choose a reason for hiding this comment

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

updated

Copy link
Contributor

@tremarkley tremarkley left a comment

Choose a reason for hiding this comment

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

left some minor comments, but looks great!


- **Embedded Wallets**: Actions supports popular embedded [wallet providers](https://actions.money/#wallet), allowing your users to access DeFi with email authentication flows alone.

- **Configure Actions**: Extend your embedded wallet with DeFi actions like Lend, Borrow, Swap, and Pay. Set multiple providers for each Action to choose the best markets across DeFi.
Copy link
Contributor

Choose a reason for hiding this comment

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

do you think we should leave out mentioning borrow, swap and pay until they're implemented?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll leave it on this line, but I'll add ⚠️ Coming Soon further down here and on those reference pages.

client: httpClient,
organizationId: session.organizationId,
signWith: walletAddress,
ethereumAddress: walletAddress,
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: the ethereumAddress argument is optional and defaults to the same value as signWith , so you could leave it out

Copy link
Contributor

Choose a reason for hiding this comment

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

removed


// Fetch Turnkey wallet
const { wallets, httpClient, session } = useTurnkey()
const embeddedWallet = wallets[0]
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: not sure if we should include this since the turnkey sdk can always change, but this is how you get an embedded wallet:

const embeddedWallet = wallets.find(
    (wallet) =>
      wallet.accounts.some(
        (account) => account.addressFormat === 'ADDRESS_FORMAT_ETHEREUM',
      ) && wallet.source === WalletSource.Embedded,
  )

you could also update the example above to use this as well instead of calling createWallet

Copy link
Contributor

Choose a reason for hiding this comment

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

added

Comment on lines +409 to +437
Configure a borrow provider with `BorrowConfig`:

```typescript
import type { BorrowConfig } from '@eth-optimism/actions-sdk'
import { USDC, ETH, WBTC } from '@eth-optimism/actions-sdk/assets'
import { USDCMorphoMarket } from './actions/markets'

const borrowConfig: BorrowConfig = {
type: 'morpho',
assetAllowlist: [USDC, ETH, WBTC],
assetBlocklist: [],
marketAllowlist: [USDCMorphoMarket],
marketBlocklist: [],
}
```

Configure a swap provider with `SwapConfig`:

```typescript
import type { SwapConfig } from '@eth-optimism/actions-sdk'
import { USDC, ETH, WBTC } from '@eth-optimism/actions-sdk/assets'

const swapConfig: SwapConfig = {
type: 'uniswap',
defaultSlippage: 100, // 100 bips or 1%
assetAllowList: [USDC, ETH, WBTC],
assetBlocklist: [],
}
```
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: it might be good to mention that these are not supported yet

Copy link
Contributor

Choose a reason for hiding this comment

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

done

@its-everdred its-everdred merged commit 904b1ca into mintlify Nov 6, 2025
7 checks passed
@its-everdred its-everdred deleted the actions branch November 6, 2025 00:49
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.

5 participants