Skip to content

Conversation

@Kamicio1234
Copy link

No description provided.

hklee93 and others added 12 commits September 25, 2023 22:51
Co-authored-by: Aurélien Schiltz <[email protected]>
* Add initial timebounds-retry implementation

* Correct syntax mistake

* Fix typo in string method name

* Add general success test case

* Correct bug with pseduo-infinite tx submission retries

* Add test for user_error_response

* Retry on 5xx, raise error on other non-success

* Add more robust user and server error tests

* Add tx submission timeout test

* Adjust comment wording

* replace hardcoded test key

* update gemspec

* Adjust timebounds variable naming

Co-authored-by: Hakkyung Lee <[email protected]>

* Remove timeout

Co-authored-by: Hakkyung Lee <[email protected]>

* Safer error code assignments for TxSubmissionError exception

Co-authored-by: Hakkyung Lee <[email protected]>

* Remove redundant imports

Co-authored-by: Hakkyung Lee <[email protected]>

* Randomly generate test wallet seed

Co-authored-by: Hakkyung Lee <[email protected]>

* Randomly generate recipient wallet address

Co-authored-by: Hakkyung Lee <[email protected]>

* Remove internal check for timeout (leaving it to Stellar SDK)

* Remove timeout test since Stellar is handling it now

* safe access

---------

Co-authored-by: Tate Mauzy <[email protected]>
Co-authored-by: Hakkyung Lee <[email protected]>
* bump version

* update validate_private_seed

* use big decimal
Copy link
Author

@Kamicio1234 Kamicio1234 left a comment

Choose a reason for hiding this comment

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

Ok

@Kamicio1234 Kamicio1234 changed the base branch from main to fix/tx-error-handling August 11, 2025 06:59
@Kamicio1234 Kamicio1234 changed the base branch from fix/tx-error-handling to feature/drop-key-from-init August 11, 2025 07:04
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.

6 participants