Skip to content

Add fee-CAT fee policy propagation across offers and storage#756

Open
ibutterbot wants to merge 2 commits intoxch-dev:mainfrom
ibutterbot:fee-cat/support
Open

Add fee-CAT fee policy propagation across offers and storage#756
ibutterbot wants to merge 2 commits intoxch-dev:mainfrom
ibutterbot:fee-cat/support

Conversation

@ibutterbot
Copy link
Copy Markdown

Summary

  • Add missing fee-policy hydration in sage-database (assets/option.rs, offers.rs, transactions.rs) so CAT fee policy survives DB reads in offer/transaction flows.
  • Propagate requested CAT fee policy through offer creation paths (RequestedCat, offer CAT hydration helper, endpoints/offers.rs, and wallet make_offer) and retain fee policy in cache/offer-summary paths.
  • Pin chia-wallet-sdk to immutable commit 7856417c6b22aa319cc6d024018f597f6fb55139, include migrations/0006_fee_cats.sql, refresh .sqlx metadata, and update fee-CAT tests to validate policy carriage/persistence.

Test plan

  • DATABASE_URL=sqlite://./test.sqlite cargo sqlx prepare --workspace -- --package sage-database --package sage-wallet --package sage
  • cargo test --package sage-database --package sage-wallet --package sage
  • cargo clippy --package sage-database --package sage-wallet --package sage
  • cargo test / cargo clippy for full workspace (blocked in this environment by missing system glib-2.0 for Tauri crates)

Made with Cursor

Carry CAT fee policy through remaining database reads, offer request hydration, and cache/summary paths so fee-CAT semantics survive import/make/view flows. Pin chia-wallet-sdk to the PR360 commit and refresh migration/sqlx metadata with focused fee-CAT policy tests.

Made-with: Cursor
Expose fee policy fields through the API and UI so fee CAT configuration can be created, validated, displayed, and included in requested offer assets.

Made-with: Cursor
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