-
Notifications
You must be signed in to change notification settings - Fork 956
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
Draft+3459 #3468
Closed
Closed
Draft+3459 #3468
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* bat/feat/list-and-offer-snapshots: rebasing tinies Added changelog
* fraccaman/limit-pgf-stewards: changelog: add #3442 cleanup bug fix: get correct storage key fix genesis files improve logs added maximum amount of stewards as genesis parameter ci: update antithesis workflow ci: update antithesis workflow
* grarco/sdk-query-height: Changelog #2891 Fallible trait bound for block height param in queries
* brent/parameterize-gas-scale: Updated example of expected string Fixes ibc e2e test fix unit test change comment on Gas Display fixes from comments changelog: add #3391 fix and clean up Light error handling remove hard-coded gas scale add gas scale to protocol params
* tomas/move-verify-shielded: changelog: add #3419 shielded_token: feature guard validation to avoid compilation into wasm move masp validation from SDK into shielded_token crate
* grarco/masp-fee-payment: Removes fallback logic when failed fee payment Renames misleading gas limit variable Removes useless write-log commit in fee payment Fixes typo Fixes masp amounts conversion Fixes broken docs Reuses token transfer Fixes typo Panics in fee payment if balance read fails Changelog #3393 Adds missing gas spending key arg to ibc tx Masp fee payment for shielded actions Fixes masp tx generation and integration tests Updates shielded wasm code to handle fee unshielding Removes unused denominate function Adds support for masp fee payment in sdk Refactors the write log api Different gas cost for storage deletes Removes write log precommit and leverages the batch log Adds integration tests for masp fee payment Refactors batch execution in case of masp fee payment Skips the execution of the first inner tx when masp fee payment Renames fee payment gas limit parameter Returns `BatchedTxResult` from masp fee payment `check_fees` drop the storage changes in case of failure `check_fees` checks masp fee payment Reworks masp fee payment to correctly handle errors. Misc refactors Passes the correct tx index to masp fee payment check Introduces masp fee payment
* grarco/early-sapling-balance-check: Extracts the sapling value balance directly in `validate_transparent_bundle` Changelog #2721 Early sapling balance check in masp vp
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3468 +/- ##
==========================================
- Coverage 53.92% 53.52% -0.41%
==========================================
Files 317 319 +2
Lines 107575 109698 +2123
==========================================
+ Hits 58011 58713 +702
- Misses 49564 50985 +1421 ☔ View full report in Codecov by Sentry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe your changes
Testing the combination of #3377 and #3459 .
Indicate on which release or other PRs this topic is based on
#3377
#3459
Checklist before merging to
draft