Skip to content

feat: process ibc ack in chain service for balance#346

Merged
mateuszjasiuk merged 3 commits intomainfrom
feat/process-ibc-ack-in-chain-service-for-balance
Apr 1, 2025
Merged

feat: process ibc ack in chain service for balance#346
mateuszjasiuk merged 3 commits intomainfrom
feat/process-ibc-ack-in-chain-service-for-balance

Conversation

@mateuszjasiuk
Copy link
Copy Markdown
Collaborator

@mateuszjasiuk mateuszjasiuk commented Mar 31, 2025

This updates balance for refund_target if ibc unshielding failed on destination chain

  • bumped namada_sdk
  • splitted get_token_and_amount into two fns
  • added ibc sender address for balance check on received ack

@mateuszjasiuk mateuszjasiuk force-pushed the feat/process-ibc-ack-in-chain-service-for-balance branch from eb9a5f5 to 6678e21 Compare March 31, 2025 11:47
@mateuszjasiuk mateuszjasiuk force-pushed the feat/process-ibc-ack-in-chain-service-for-balance branch from 6678e21 to a08231c Compare March 31, 2025 11:50
@mateuszjasiuk mateuszjasiuk requested review from Fraccaman, grarco and sug0 and removed request for sug0 March 31, 2025 11:53
Copy link
Copy Markdown
Collaborator

@sug0 sug0 left a comment

Choose a reason for hiding this comment

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

left a single comment (ignore the other one about code style tbh)

Comment thread shared/src/utils.rs
Comment thread shared/src/utils.rs Outdated
@mateuszjasiuk mateuszjasiuk requested a review from sug0 April 1, 2025 13:43
Comment thread shared/src/utils.rs
@mateuszjasiuk mateuszjasiuk enabled auto-merge April 1, 2025 14:02
@mateuszjasiuk mateuszjasiuk added this pull request to the merge queue Apr 1, 2025
Merged via the queue into main with commit e0e4ae9 Apr 1, 2025
14 checks passed
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.

3 participants