-
Notifications
You must be signed in to change notification settings - Fork 15
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
refactor/PRO-1101: rid of swapping minimum swap amount #4419
refactor/PRO-1101: rid of swapping minimum swap amount #4419
Conversation
--> state-chain/pallets/cf-swapping/src/lib.rs:395:5 --> state-chain/pallets/cf-swapping/src/lib.rs:698:4 --> state-chain/pallets/cf-swapping/src/lib.rs:700:36 --> state-chain/pallets/cf-swapping/src/lib.rs:746:29 --> state-chain/pallets/cf-swapping/src/lib.rs:750:31 --> state-chain/pallets/cf-swapping/src/lib.rs:974:16 --> state-chain/pallets/cf-swapping/src/lib.rs:980:37 --> state-chain/pallets/cf-swapping/src/lib.rs:247:11 --> state-chain/pallets/cf-swapping/src/lib.rs:328:4 --> state-chain/pallets/cf-swapping/src/lib.rs:329:4 --> state-chain/pallets/cf-swapping/src/lib.rs:333:4 --> state-chain/pallets/cf-swapping/src/lib.rs:334:4 --> state-chain/pallets/cf-swapping/src/lib.rs:335:4 --> state-chain/pallets/cf-swapping/src/lib.rs:336:4 --> state-chain/pallets/cf-swapping/src/lib.rs:394:25 --> state-chain/pallets/cf-swapping/src/lib.rs:402:11 --> state-chain/pallets/cf-swapping/src/lib.rs:692:10 --> state-chain/pallets/cf-swapping/src/lib.rs:372:3 --> state-chain/node/src/chain_spec.rs:675:30
22350a4
to
846f443
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #4419 +/- ##
======================================
- Coverage 72% 72% -0%
======================================
Files 396 396
Lines 67561 66981 -580
Branches 67561 66981 -580
======================================
- Hits 48756 48153 -603
- Misses 16363 16392 +29
+ Partials 2442 2436 -6 ☔ View full report in Codecov by Sentry. |
…of-swapping-MinimumSwapAmount
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good from product side
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a few nitty comments.
d6338d9
to
fd0030e
Compare
…itness-swap * origin/main: (24 commits) fix: restore correct restriction on redemption expiry (PRO-1072) Feat/migrate-to-polkadot-sdk-repo (#4361) chore: fix RUSTSEC-2024-0003 (#4426) Feat: decouple CFE from SC events (#4382) chore: add docker tag prefix to `chainflip-ingress-egress-tacker` 🏷️ (#4427) refactor/PRO-1101: rid of swapping minimum swap amount (#4419) refactor(ingress-egress-tracker): remove unnecessary fields (#4425) Improved the retry queue storage (#4420) fix: bump spec version command only bumps when necessary (#4422) refactor: use yargs for all try_runtime_command options (#4423) fix: await finalisation before starting broker (#4412) chore: debug logs to see get_raw_txs query (#4413) doc: correct env vars (#4416) fix: pool_orders rpc filters empty orders (PRO-1039) (#4377) Produce an event in case the swap yields a zero egress amount (#4410) fix: don't have conflicting redis port with localnet (#4415) Ability to specify output for the subcommands, other than `/dev/null` (#4411) chore: increase limit on max number of bitcoin payloads in a ceremony to theoretical maximum (#4396) refactor(ingress-egress-tracker): configurable expiry times [WEB-761] (#4406) fix: use existing script for upgrade job (#4403) ... # Conflicts: # Cargo.lock # state-chain/primitives/Cargo.toml
Pull Request
Closes: PRO-1101
Checklist
Please conduct a thorough self-review before opening the PR.
Summary
TBD