Skip to content
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

[ETHEREUM-CONTRACTS] use registerAppWithKey #1554

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

0xdavinchee
Copy link
Contributor

use registerAppWithKey instead of registerApp as the latter is deprecated

@0xdavinchee 0xdavinchee requested a review from hellwolf as a code owner July 31, 2023 12:55
@github-actions
Copy link

Changelog Reminder

Reminder to update the CHANGELOG.md for any of the modified packages in this PR.

  • CHANGELOG.md modified
  • Double check before merge

@codecov
Copy link

codecov bot commented Jul 31, 2023

Codecov Report

Patch coverage has no change and project coverage change: +8.81% 🎉

Comparison is base (1ef77a5) 87.53% compared to head (fcc658f) 96.35%.
Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1554      +/-   ##
==========================================
+ Coverage   87.53%   96.35%   +8.81%     
==========================================
  Files          91       41      -50     
  Lines        4389     2139    -2250     
  Branches      184        0     -184     
==========================================
- Hits         3842     2061    -1781     
+ Misses        503       78     -425     
+ Partials       44        0      -44     
Flag Coverage Δ
ethereum-contracts 96.35% <0.00%> (ø)
sdk-core ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
...reum-contracts/contracts/apps/SuperAppBaseFlow.sol 0.00% <0.00%> (ø)
...reum-contracts/contracts/superfluid/Superfluid.sol 97.44% <ø> (ø)

... and 50 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -28,7 +28,8 @@ abstract contract SuperAppBaseFlow is ISuperApp {
ISuperfluid host_,
bool activateOnCreated,
bool activateOnUpdated,
bool activateOnDeleted
bool activateOnDeleted,
string memory registrationKey
Copy link
Contributor

Choose a reason for hiding this comment

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

breaking change..., but let's do it

@hellwolf
Copy link
Contributor

oh codecov says no :) @0xdavinchee

@0xdavinchee
Copy link
Contributor Author

oh codecov says no :) @0xdavinchee

codecov is being a bully :(

@0xdavinchee 0xdavinchee merged commit f94149c into dev Jul 31, 2023
@0xdavinchee 0xdavinchee deleted the fix-register-app-superapp-base-flow branch July 31, 2023 18:35
@github-actions
Copy link

XKCD Comic Relif

Link: https://xkcd.com/1554
https://xkcd.com/1554

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