Skip to content

Commit

Permalink
Remove NotificationController (#4815)
Browse files Browse the repository at this point in the history
Removes the `NotificationController` in favor of
`NotificationServicesController`, see
#4809
  • Loading branch information
hmalik88 authored Nov 29, 2024
1 parent 9857fad commit 60e3ce7
Show file tree
Hide file tree
Showing 16 changed files with 3 additions and 740 deletions.
3 changes: 0 additions & 3 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@
/packages/phishing-controller @MetaMask/product-safety

## Snaps Team
/packages/notification-controller @MetaMask/snaps-devs
/packages/rate-limit-controller @MetaMask/snaps-devs

## Wallet API Platform Team
Expand Down Expand Up @@ -86,8 +85,6 @@
/packages/message-manager/CHANGELOG.md @MetaMask/confirmations @MetaMask/wallet-framework-engineers
/packages/name-controller/package.json @MetaMask/confirmations @MetaMask/wallet-framework-engineers
/packages/name-controller/CHANGELOG.md @MetaMask/confirmations @MetaMask/wallet-framework-engineers
/packages/notification-controller/package.json @MetaMask/snaps-devs @MetaMask/wallet-framework-engineers
/packages/notification-controller/CHANGELOG.md @MetaMask/snaps-devs @MetaMask/wallet-framework-engineers
/packages/notification-services-controller/package.json @MetaMask/notifications @MetaMask/wallet-framework-engineers
/packages/notification-services-controller/CHANGELOG.md @MetaMask/notifications @MetaMask/wallet-framework-engineers
/packages/phishing-controller/package.json @MetaMask/product-safety @MetaMask/wallet-framework-engineers
Expand Down
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ Each package in this repository has its own README where you can find installati
- [`@metamask/multichain`](packages/multichain)
- [`@metamask/name-controller`](packages/name-controller)
- [`@metamask/network-controller`](packages/network-controller)
- [`@metamask/notification-controller`](packages/notification-controller)
- [`@metamask/notification-services-controller`](packages/notification-services-controller)
- [`@metamask/permission-controller`](packages/permission-controller)
- [`@metamask/permission-log-controller`](packages/permission-log-controller)
Expand Down Expand Up @@ -85,7 +84,6 @@ linkStyle default opacity:0.5
multichain(["@metamask/multichain"]);
name_controller(["@metamask/name-controller"]);
network_controller(["@metamask/network-controller"]);
notification_controller(["@metamask/notification-controller"]);
notification_services_controller(["@metamask/notification-services-controller"]);
permission_controller(["@metamask/permission-controller"]);
permission_log_controller(["@metamask/permission-log-controller"]);
Expand Down Expand Up @@ -141,7 +139,6 @@ linkStyle default opacity:0.5
network_controller --> controller_utils;
network_controller --> eth_json_rpc_provider;
network_controller --> json_rpc_engine;
notification_controller --> base_controller;
notification_services_controller --> base_controller;
notification_services_controller --> controller_utils;
notification_services_controller --> keyring_controller;
Expand Down
175 changes: 0 additions & 175 deletions packages/notification-controller/CHANGELOG.md

This file was deleted.

20 changes: 0 additions & 20 deletions packages/notification-controller/LICENSE

This file was deleted.

15 changes: 0 additions & 15 deletions packages/notification-controller/README.md

This file was deleted.

26 changes: 0 additions & 26 deletions packages/notification-controller/jest.config.js

This file was deleted.

71 changes: 0 additions & 71 deletions packages/notification-controller/package.json

This file was deleted.

Loading

0 comments on commit 60e3ce7

Please sign in to comment.