Skip to content

Rework redeem multiplier to bps#183

Open
evgeny-stakewise wants to merge 3 commits intomasterfrom
redeem-multiplier
Open

Rework redeem multiplier to bps#183
evgeny-stakewise wants to merge 3 commits intomasterfrom
redeem-multiplier

Conversation

@evgeny-stakewise
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 26, 2026 17:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reworks the osToken redemption multiplier representation to use an integer basis-points (bps) field while keeping a derived Decimal multiplier available via a property on ProtocolConfig.

Changes:

  • Replace os_token_redeem_multiplier: Decimal with os_token_redeem_multiplier_bps: int and add a computed os_token_redeem_multiplier property.
  • Update protocol config building and test factories to use the new _bps field/key and default.
  • Bump package version and extend .gitignore.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
sw_utils/typings.py Renames multiplier field to bps and adds a computed Decimal property.
sw_utils/protocol_config.py Reads multiplier from os_token_redeem_multiplier_bps config key and passes into ProtocolConfig.
sw_utils/tests/factories.py Updates test factory defaults and construction to the new bps field.
pyproject.toml Version bump v0.12.9v0.12.10.
.gitignore Ignores CLAUDE.md and .claude.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants