Skip to content

feat(api): include tokens in approval transactions#1239

Draft
oXtxNt9U wants to merge 5 commits intodevelopfrom
feat/api/token-approvals
Draft

feat(api): include tokens in approval transactions#1239
oXtxNt9U wants to merge 5 commits intodevelopfrom
feat/api/token-approvals

Conversation

@oXtxNt9U
Copy link
Contributor

@oXtxNt9U oXtxNt9U commented Mar 3, 2026

Summary

#1236 added the tokens field to transactions when tokens are transferred.

This PR extends it to also cover approval transactions.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.60%. Comparing base (feefb65) to head (6f7844f).

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #1239       +/-   ##
============================================
+ Coverage    53.09%   98.60%   +45.50%     
============================================
  Files         1121        3     -1118     
  Lines        62313      430    -61883     
  Branches      1700       77     -1623     
============================================
- Hits         33088      424    -32664     
+ Misses       29070        4    -29066     
+ Partials       155        2      -153     
Flag Coverage Δ
contracts 98.60% <ø> (?)
packages ?

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant