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: Allow contract-to-contract write methods #550

Merged
merged 10 commits into from
Sep 30, 2024

Conversation

AgustinRamiroDiaz
Copy link
Contributor

Fixes #454

What

Allows contract to contract write operations

Why

To allow more types of contracts

Testing done

Added end to end test

Questions

Should we somehow return the created transactions to the user? I mean, maybe they would require them to track it somehow

Maybe as metadata in the main transaction? something like triggered transactions?

Checks

  • I have tested this code
  • I have reviewed my own PR
  • I have created an issue for this PR
  • I have set a descriptive PR title compliant with conventional commits

User facing release notes

Now you can call write methods from other contracts from your Intelligent Contract!

Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
@AgustinRamiroDiaz AgustinRamiroDiaz linked an issue Sep 27, 2024 that may be closed by this pull request
Copy link

codecov bot commented Sep 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.32%. Comparing base (72577e0) to head (3524602).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
+ Coverage   16.28%   16.32%   +0.03%     
==========================================
  Files         111      111              
  Lines        7840     7843       +3     
  Branches      184      184              
==========================================
+ Hits         1277     1280       +3     
  Misses       6488     6488              
  Partials       75       75              

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

Copy link

sonarcloud bot commented Sep 30, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@AgustinRamiroDiaz AgustinRamiroDiaz marked this pull request as ready for review September 30, 2024 12:20
@AgustinRamiroDiaz AgustinRamiroDiaz merged commit 6a6b70a into main Sep 30, 2024
14 of 15 checks passed
Copy link
Contributor

github-actions bot commented Oct 1, 2024

🎉 This PR is included in version 0.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@cristiam86 cristiam86 deleted the 454-contract-to-contract-write branch October 16, 2024 15:08
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.

Contract-to-contract write
2 participants