Skip to content

chore: address all v5 todos left in code#2380

Merged
jakehobbs merged 3 commits intov5.x.xfrom
jake/v5-todos
Mar 9, 2026
Merged

chore: address all v5 todos left in code#2380
jakehobbs merged 3 commits intov5.x.xfrom
jake/v5-todos

Conversation

@jakehobbs
Copy link
Collaborator

@jakehobbs jakehobbs commented Mar 5, 2026

Pull Request Checklist


PR-Codex overview

This PR focuses on the removal of deprecated decorators and actions related to multi-owner and single-owner accounts in the smart-accounts package, along with updates to documentation and code for improved clarity and functionality.

Detailed summary

  • Deleted files related to multiOwner and singleOwner decorators.
  • Updated JSDoc comments for clarity, removing TODOs.
  • Refactored sendUserOperation calls in tests to use encode methods.
  • Adjusted type definitions and documentation references for consistency.
  • Removed unused exports for actions in the public interface.
  • Cleaned up test cases related to multi-owner accounts.

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

@vercel
Copy link

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aa-sdk-ui-demo Error Error Mar 9, 2026 4:41pm

Request Review

Copy link
Collaborator

@0xfourzerofour 0xfourzerofour left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@0xfourzerofour 0xfourzerofour left a comment

Choose a reason for hiding this comment

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

LGTM

@jakehobbs jakehobbs merged commit f5525e7 into v5.x.x Mar 9, 2026
10 of 11 checks passed
@jakehobbs jakehobbs deleted the jake/v5-todos branch March 9, 2026 16:47
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