-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor error handling page #649
Conversation
aditya1702
commented
Jun 4, 2024
•
edited
Loading
edited
- Updates the error-handling page: https://developers.stellar.org/docs/learn/encyclopedia/error-handling. Adds information about error handling strategies for async transaction submission endpoint.
- Update the error handling strategies for non-async transaction submission: On 504 timeout, instead of recommending users to always retry submissions, redirect them to check for the tx status and then retry if the txn is not included in a ledger. Simultaneous PR to add the tx hash to timeout response: services/horizon: Add hash to the txsub timeout response go#5328
- Moves the page from encyclopedia to admin-guide in Horizon docs.
Something went wrong with PR preview build please check |
Something went wrong with PR preview build please check |
Preview is available here: |
Co-authored-by: Jake Urban <[email protected]>
Preview is available here: |
Co-authored-by: Jake Urban <[email protected]>
Preview is available here: |
Co-authored-by: Jake Urban <[email protected]>
Preview is available here: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! I did a quick first pass review
Co-authored-by: Molly Karcher <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Preview is available here: |
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Co-authored-by: George <[email protected]>
Something went wrong with PR preview build please check |
Preview is available here: |
6 similar comments
Preview is available here: |
Preview is available here: |
Preview is available here: |
Preview is available here: |
Preview is available here: |
Preview is available here: |
Co-authored-by: George <[email protected]>
# Conflicts: # docs/build/guides/basics/create-account.mdx # docs/build/guides/basics/send-and-receive-payments.mdx
Preview is available here: |
Something went wrong with PR preview build please check |
Preview is available here: |
Preview is available here: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 👏 👏 great effort on this!!
Preview is available here: |