Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

chore: add deprecation notices to the AccountsController#8027

Merged
hmalik88 merged 5 commits intomain from
hm/mul-1264
Feb 24, 2026
Merged

chore: add deprecation notices to the AccountsController #8027
hmalik88 merged 5 commits intomain from
hm/mul-1264

Conversation

@hmalik88
Copy link
Contributor

@hmalik88 hmalik88 commented Feb 24, 2026
edited by cursor bot
Loading

Explanation

The AccountsController will soon be phased out and so deprecation notices are being added across the class, methods and types to discourage use and inform of the new way of interacting with accounts.

References

N/A

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Doc-only changes (JSDoc @deprecated annotations and changelog entry) with no runtime behavior changes; risk is limited to consumers treating deprecations as warnings/errors in stricter builds.

Overview
Marks AccountsController as deprecated across the package by adding JSDoc @deprecated annotations to the class, EMPTY_ACCOUNT, exported action/event/state types, and key public methods (e.g., getAccount, listMultichainAccounts, updateAccounts, loadBackup).

Updates the accounts-controller changelog to announce the deprecation and point consumers to AccountTreeController, MultichainAccountService, and Keyring API v2.

Written by Cursor Bugbot for commit 3e3f768. This will update automatically on new commits. Configure here.

@hmalik88 hmalik88 requested review from a team as code owners February 24, 2026 03:34
@hmalik88 hmalik88 added this pull request to the merge queue Feb 24, 2026
Merged via the queue into main with commit 45920af Feb 24, 2026
310 checks passed
@hmalik88 hmalik88 deleted the hm/mul-1264 branch February 24, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mathieuartu mathieuartu mathieuartu approved these changes

@gantunesr gantunesr gantunesr approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /