Skip to content

Update masp events#331

Closed
grarco wants to merge 9 commits intomainfrom
grarco/update-masp-events
Closed

Update masp events#331
grarco wants to merge 9 commits intomainfrom
grarco/update-masp-events

Conversation

@grarco
Copy link
Copy Markdown
Contributor

@grarco grarco commented Mar 8, 2025

Depends on namada-net/namada#4449 -> namada-net/namada#4475.

  • Updates the logic that fetches masp events to match the new event format
  • Extends the WrapperTx with the information on whether it ran fee payment via the shielded pool
  • Improves the logic to check the success of an inner tx to take into account masp fee payment

@grarco grarco mentioned this pull request Mar 8, 2025
3 tasks
@grarco grarco force-pushed the grarco/update-masp-events branch from 8c39eb8 to ca94488 Compare March 8, 2025 14:28
@grarco grarco marked this pull request as draft March 8, 2025 15:38
@grarco grarco marked this pull request as ready for review March 10, 2025 10:28
@grarco grarco requested review from Fraccaman and sug0 March 10, 2025 10:28
Comment thread shared/src/block_result.rs Outdated
Comment thread shared/src/transaction.rs
@Fraccaman
Copy link
Copy Markdown
Collaborator

merged via #354

@Fraccaman Fraccaman closed this May 2, 2025
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