feat: lower limit-order placement fees one tier (tier 9 is 0) #1058

Merged

PlasticDigits merged 2 commits from feat/514-limit-order-tier-fees into main

2026-08-15 21:57:23 +00:00

PlasticDigits commented

2026-08-15 21:50:08 +00:00

(Migrated from gitlab.com)

Summary

Shift each self-register tier’s limit-order placement discount one step down the remaining-fee ladder; tier 9 placement is 0 bps. CL8Y hold minima are unchanged (tier 9 still 7,500 CL8Y).
Swap and book-take fees stay on the current discount_bps ladder, so crossing the book still charges the taker’s standard trading fee (unregistered @ 180 bps pair = 90 bps taker). Confirmed current fill behavior: maker half at place, taker half at fill.
New optional limit_discount_bps on AddTier / UpdateTier / GetDiscount. Omitted → placement equals swap discount (custom test tiers unchanged). Production deploys set the shifted values. Fee-discount migrate 1.1.0 backfills standard tier IDs; pair 1.12.0 uses the limit discount on place.

Closes #514

Test plan

make verify-issue-514 (unit + integration + docs drift + frontend + upgrade DRY_RUN; includes retest)
After LocalTerra / mainnet upgrade: GetTiers shows tier 9 limit_discount_bps = 10000 and discount_bps = 9500
Registered tier-9 wallet: place limit → maker_fee_amount = 0; same wallet pool/hybrid swap → effective_fee_bps = 9 (180 bps pair)
Unregistered place still 90 bps maker; unregistered take of a resting order still 90 bps taker
/tiers “Limit place” column and Trade pre-submit maker-fee copy match the new schedule
Mainnet: run scripts/upgrade-514-limit-discount.sh (store + migrate fee-discount, then all pairs)

Closes #514

## Summary - Shift each self-register tier’s **limit-order placement** discount one step down the remaining-fee ladder; **tier 9 placement is 0 bps**. CL8Y hold minima are unchanged (tier 9 still 7,500 CL8Y). - Swap and book-**take** fees stay on the current `discount_bps` ladder, so crossing the book still charges the taker’s standard trading fee (unregistered @ 180 bps pair = 90 bps taker). Confirmed current fill behavior: maker half at place, taker half at fill. - New optional `limit_discount_bps` on `AddTier` / `UpdateTier` / `GetDiscount`. Omitted → placement equals swap discount (custom test tiers unchanged). Production deploys set the shifted values. Fee-discount migrate **1.1.0** backfills standard tier IDs; pair **1.12.0** uses the limit discount on place. Closes #514 ## Test plan - [ ] `make verify-issue-514` (unit + integration + docs drift + frontend + upgrade DRY_RUN; includes retest) - [ ] After LocalTerra / mainnet upgrade: `GetTiers` shows tier 9 `limit_discount_bps` = 10000 and `discount_bps` = 9500 - [ ] Registered tier-9 wallet: place limit → `maker_fee_amount` = 0; same wallet pool/hybrid swap → `effective_fee_bps` = 9 (180 bps pair) - [ ] Unregistered place still 90 bps maker; unregistered take of a resting order still 90 bps taker - [ ] `/tiers` “Limit place” column and Trade pre-submit maker-fee copy match the new schedule - [ ] Mainnet: run `scripts/upgrade-514-limit-discount.sh` (store + migrate fee-discount, then all pairs) Closes #514

PlasticDigits commented

2026-08-15 21:50:15 +00:00

(Migrated from gitlab.com)

marked this merge request as ready

marked this merge request as **ready**

PlasticDigits commented

2026-08-15 21:50:23 +00:00

(Migrated from gitlab.com)

mentioned in issue #514

PlasticDigits commented

2026-08-15 21:57:18 +00:00

(Migrated from gitlab.com)

added 2 commits

1f254d4e - 1 commit from branch main
64842c59 - Merge remote-tracking branch 'origin/main' into feat/514-limit-order-tier-fees

Compare with previous version

added 2 commits <ul><li>1f254d4e - 1 commit from branch <code>main</code></li><li>64842c59 - Merge remote-tracking branch 'origin/main' into feat/514-limit-order-tier-fees</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/340/diffs?diff_id=1964028540&start_sha=35c2a7cc05ee4863455ee0eb2975446e75648f73)