Skip to content

Conversation

@yrong
Copy link
Contributor

@yrong yrong commented Sep 10, 2025

Context

This allows users to pay additional fees to speed up transfers using the FiatShamir approach. More context in PR-1462

The gas cost is benchmarked here.

Further changes are required on the relay side to pick up transfers/orders using the FiatShamir approach.

@yrong yrong changed the base branch from main to ron/submitFiatShamir September 10, 2025 08:41
@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.10%. Comparing base (ee36155) to head (3e62355).

Additional details and impacted files
@@                  Coverage Diff                  @@
##           ron/submitFiatShamir    #1562   +/-   ##
=====================================================
  Coverage                 80.10%   80.10%           
=====================================================
  Files                        21       21           
  Lines                       930      930           
  Branches                    162      162           
=====================================================
  Hits                        745      745           
  Misses                      159      159           
  Partials                     26       26           
Flag Coverage Δ
solidity 80.10% <ø> (ø)

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.

@vgeddes vgeddes mentioned this pull request Sep 21, 2025
Base automatically changed from ron/submitFiatShamir to main November 28, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants