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

Feat(ACI): Implement Contract Events #893

Merged
merged 14 commits into from
Feb 11, 2020
Merged

Feat(ACI): Implement Contract Events #893

merged 14 commits into from
Feb 11, 2020

Conversation

nduchak
Copy link
Contributor

@nduchak nduchak commented Feb 4, 2020

No description provided.

@nduchak nduchak added the feature label Feb 4, 2020
@nduchak nduchak added this to the 7.1.0 milestone Feb 4, 2020
@nduchak nduchak self-assigned this Feb 4, 2020
@nduchak nduchak changed the title Feat(ACI): Implement Contract Events [WIP]Feat(ACI): Implement Contract Events Feb 4, 2020
@nduchak
Copy link
Contributor Author

nduchak commented Feb 4, 2020

Resolve #428

@todo: investigate and handle all types allowed in event body
@lgtm-com
Copy link

lgtm-com bot commented Feb 4, 2020

This pull request introduces 1 alert when merging ff25b10 into 358e673 - view on LGTM.com

new alerts:

  • 1 for Superfluous trailing arguments

@codecov
Copy link

codecov bot commented Feb 4, 2020

Codecov Report

Merging #893 into develop will increase coverage by 0.11%.
The diff coverage is 92.85%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #893      +/-   ##
===========================================
+ Coverage    92.86%   92.98%   +0.11%     
===========================================
  Files           57       57              
  Lines         2846     2880      +34     
===========================================
+ Hits          2643     2678      +35     
+ Misses         203      202       -1
Impacted Files Coverage Δ
es/contract/aci/helpers.js 100% <ø> (ø) ⬆️
es/chain/node.js 94.11% <100%> (ø) ⬆️
es/contract/aci/index.js 95.65% <100%> (+0.41%) ⬆️
es/contract/aci/transformation.js 96.22% <90.9%> (-0.86%) ⬇️
es/channel/handlers.js 79.72% <0%> (-0.46%) ⬇️
es/contract/compiler.js 100% <0%> (ø) ⬆️
es/utils/aepp-wallet-communication/rpc/aepp-rpc.js 100% <0%> (ø) ⬆️
es/accounts.js 100% <0%> (ø) ⬆️
es/node.js 96.96% <0%> (ø) ⬆️
es/node-pool/index.js 100% <0%> (ø) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 358e673...d06692b. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Feb 5, 2020

This pull request introduces 1 alert when merging ad47035 into 358e673 - view on LGTM.com

new alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Feb 5, 2020

This pull request introduces 1 alert when merging 20ea4d0 into 358e673 - view on LGTM.com

new alerts:

  • 1 for Superfluous trailing arguments

@mradkov mradkov self-requested a review February 7, 2020 12:53
Add test's for Events decoding without ACI
Adjust `contract-events` guide
Minor ACI validation improvement
@nduchak nduchak reopened this Feb 11, 2020
@nduchak nduchak changed the title [WIP]Feat(ACI): Implement Contract Events Feat(ACI): Implement Contract Events Feb 11, 2020
@mradkov mradkov self-requested a review February 11, 2020 14:41
@nduchak nduchak merged commit cce7899 into develop Feb 11, 2020
@nduchak nduchak deleted the feat/contract-events branch February 11, 2020 14:49
@mradkov mradkov mentioned this pull request Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants