Skip to content

fix: thor token repayments regression#6698

Merged
gomesalexandre merged 3 commits intodevelopfrom
fix_thor_token_repayments
Apr 16, 2024
Merged

fix: thor token repayments regression#6698
gomesalexandre merged 3 commits intodevelopfrom
fix_thor_token_repayments

Conversation

@gomesalexandre
Copy link
Copy Markdown
Contributor

@gomesalexandre gomesalexandre commented Apr 16, 2024

Description

See #6676 for the original PR, with this PR fixing the spotted bug in 1.574.0 (https://discord.com/channels/554694662431178782/1229606430173036546/1229636186822283386)

Pull Request Type

  • 🐛 Bug fix (Non-breaking Change: Fixes an issue)
  • 🛠️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
  • 💅 New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

N/A

Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Low - ensure this time around, this indeed doesn't bring regression on non-token repayments, which I'm still unable to test because I don't have repayable position yet

Testing

  • There are no regressions when repaying a position using a non-native token
  • Additionally, if you have two repayable positions, ensure repaying with a token is happy

Engineering

  • ☝🏽

Operations

  • ☝🏽

Screenshots (if applicable)

@gomesalexandre gomesalexandre requested a review from a team as a code owner April 16, 2024 07:49
@gomesalexandre gomesalexandre marked this pull request as draft April 16, 2024 07:50
@gomesalexandre gomesalexandre changed the title fix: revert of the revert of thor token repayments fix: thor token repayments regression Apr 16, 2024
@gomesalexandre gomesalexandre marked this pull request as ready for review April 16, 2024 07:57
Copy link
Copy Markdown
Member

@0xApotheosis 0xApotheosis left a comment

Choose a reason for hiding this comment

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

Ah yes, that'll do it!

@gomesalexandre gomesalexandre merged commit 44660c3 into develop Apr 16, 2024
@gomesalexandre gomesalexandre deleted the fix_thor_token_repayments branch April 16, 2024 08:37
0xApotheosis added a commit that referenced this pull request Apr 17, 2024
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