-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: STAKE-934 add earn controller to engine #13778
base: main
Are you sure you want to change the base?
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
New dependencies detected. Learn more about Socket for GitHub ↗︎
Have feedback? Participate in our User Experience Survey 📊 |
c1977d8
to
a85b3d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
b500f68
to
93c4ec5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13778 +/- ##
==========================================
+ Coverage 63.42% 63.54% +0.12%
==========================================
Files 2093 2097 +4
Lines 45277 45316 +39
Branches 6234 6243 +9
==========================================
+ Hits 28717 28797 +80
+ Misses 14676 14637 -39
+ Partials 1884 1882 -2 ☔ View full report in Codecov by Sentry. |
074b7a7
to
e2fc165
Compare
|
Description
This PR adds the @metamask/earn-controller dependency and follows the modular controller init to add it to the Engine.
Related issues
Manual testing steps
Screenshots/Recordings
N/A
Before
N/A
After
N/A
Pre-merge author checklist
Pre-merge reviewer checklist