Skip to content
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

Expiration Entry updates and return core-generated DiagnosticEvents on the tx submission endpoint #3898

Merged
merged 13 commits into from
Aug 31, 2023

Conversation

graydon
Copy link
Contributor

@graydon graydon commented Aug 29, 2023

Implementation of this plan. Needs to not land until the corresponding XDR lands, and then env is updated, and then core is updated with XDR and env.

@graydon graydon requested a review from sisuresh August 29, 2023 06:04
Copy link
Contributor

@sisuresh sisuresh left a comment

Choose a reason for hiding this comment

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

Thanks! Just had one comment about adding more errors.

@graydon graydon force-pushed the tx-submit-diagnostics branch 3 times, most recently from f25895d to 890394b Compare August 30, 2023 06:53
@graydon
Copy link
Contributor Author

graydon commented Aug 30, 2023

Updated to be in line behind #3899 -- I'll review that tomorrow, it's a little larger than I've got time for tonight.

@MonsieurNicolas
Copy link
Contributor

r+ 1464cec

@MonsieurNicolas MonsieurNicolas changed the title Add plumbing for returning core-generated DiagnosticEvents on the tx? submission endpoint Expiration Entry updates and return core-generated DiagnosticEvents on the tx submission endpoint Aug 31, 2023
@MonsieurNicolas
Copy link
Contributor

updated the title as to reflect the actual scope of the PR

@MonsieurNicolas
Copy link
Contributor

r+ 937e092

@SirTyson SirTyson mentioned this pull request Aug 31, 2023
6 tasks
@latobarita latobarita merged commit 22b9bb3 into stellar:master Aug 31, 2023
8 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.

5 participants