Skip to content

fix: include account in call to estimateGas - #253

Merged
0xRAG merged 1 commit into
masterfrom
AGENT-115
Feb 4, 2025
Merged

0xRAG merged 1 commit into
masterfrom
AGENT-115

Conversation

@0xRAG

@0xRAG 0xRAG commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

What changed? Why?

Closes #238

Qualified Impact

@cb-heimdall

cb-heimdall commented Feb 4, 2025 •

Copy link
Copy Markdown

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 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 0
Sum 1

@0xRAG
0xRAG merged commit 887399f into master Feb 4, 2025
@0xRAG
0xRAG deleted the AGENT-115 branch February 4, 2025 17:49
emiridbest pushed a commit to emiridbest/agentkit that referenced this pull request Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Call to estimateGas in prepareTransaction of CdpWalletProvider uses address 0

3 participants