fix: Fix custom priority pools order bug - Githubissues

NexusMutual / cover-router

Computes optimal capacity allocation per Nexus Mutual staking pool on cover purchase

MIT License

1 stars 1 forks source link

fix: Fix custom priority pools order bug #103

Closed rackstar closed 1 week ago

rackstar commented 1 week ago

Context

it mutates the reducer state so subsequent calls have empty custom priority pools array which causes it to fail

Changes proposed in this pull request

fix bug

Test plan

manual test locally

Checklist

[x] Rebased the base branch
[ ] Attached corresponding Github issue
[x] Prefixed the name with the type of change (i.e. feat, chore, test)
[ ] Performed a self-review of my own code
[ ] Followed the style guidelines of this project
[ ] Made corresponding changes to the documentation
[ ] Didn't generate new warnings
[ ] Didn't generate failures on existing tests
[ ] Added tests that prove my fix is effective or that my feature works

Review

When reviewing a PR, please indicate intention in comments using the following emojis:

:cake: = Nice to have but not essential.
:bulb: = Suggestion or a comment based on personal opinion
:hammer: = I believe this should be changed.
:thinking: = I don’t understand something, do you mind giving me more context?
:rocket: = Feedback