Skip to content

Reorder event call to emit correct accountAddress in AccountCreated event in AccountFactory example#48

Merged
jarrodwatts merged 1 commit intoAbstract-Foundation:mainfrom
minebuu:patch-1
Jul 8, 2025
Merged

Reorder event call to emit correct accountAddress in AccountCreated event in AccountFactory example#48
jarrodwatts merged 1 commit intoAbstract-Foundation:mainfrom
minebuu:patch-1

Conversation

@minebuu
Copy link
Contributor

@minebuu minebuu commented Jun 17, 2025

Reordered AccountCreated event call to ensure the AccountCreated event emits the correct accountAddress.


PR-Codex overview

This PR focuses on the AccountFactory contract by ensuring that the AccountCreated event is emitted correctly after a new account is deployed.

Detailed summary

  • Added the emission of the AccountCreated event with accountAddress and owner after successful deployment.
  • Declared the AccountCreated event with indexed parameters for accountAddress and owner.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Reordered AccountCreated event call to ensure the AccountCreated event emits the correct accountAddress.
@vercel
Copy link

vercel bot commented Jun 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agw-connectkit-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2025 4:34am
agw-eoa-linking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2025 4:34am
agw-rainbowkit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2025 4:34am
agw-signing-messages ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2025 4:34am
session-keys ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2025 4:34am

@vercel
Copy link

vercel bot commented Jun 17, 2025

@minebuu is attempting to deploy a commit to the Abstract Foundation Team on Vercel.

A member of the Team first needs to authorize it.

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.

2 participants