Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Fix for missing fee bump result (issue 388) #394

Merged

Conversation

jopmiddelkamp
Copy link
Collaborator

@jopmiddelkamp jopmiddelkamp commented Jun 19, 2023

Fix for issue 388.

Also fixed the fee bump fee according to the Stellar documentation.
https://developers.stellar.org/docs/encyclopedia/fee-bump-transactions#fee

It states that the fee set on the fee bump transaction should be the amount of stopen that you would like to pay for executing only the fee bump operation which is 1 operation. So this should be independent from the inner transaction.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@jopmiddelkamp jopmiddelkamp force-pushed the feature/fee_bump_result_support branch 2 times, most recently from 7651376 to c20873e Compare June 20, 2023 08:41
@jopmiddelkamp jopmiddelkamp force-pushed the feature/fee_bump_result_support branch from c20873e to 0de588b Compare June 20, 2023 08:45
@jopmiddelkamp
Copy link
Collaborator Author

@elucidsoft, could you check this PR?

@jopmiddelkamp jopmiddelkamp force-pushed the feature/fee_bump_result_support branch 2 times, most recently from 2e5faa1 to effa08c Compare June 20, 2023 09:16
@jopmiddelkamp jopmiddelkamp force-pushed the feature/fee_bump_result_support branch from effa08c to 2401341 Compare June 20, 2023 09:20
Copy link
Owner

@elucidsoft elucidsoft left a comment

Choose a reason for hiding this comment

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

LGTM

@elucidsoft elucidsoft merged commit 6972ad6 into elucidsoft:master Jun 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants