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

feat: adding events metadata #37

Merged
merged 1 commit into from
Sep 13, 2024
Merged

feat: adding events metadata #37

merged 1 commit into from
Sep 13, 2024

Conversation

matiwinnetou
Copy link
Collaborator

No description provided.

@matiwinnetou matiwinnetou changed the title feat: adding event metadata. feat: adding events metadata Sep 11, 2024
@matiwinnetou matiwinnetou force-pushed the refakt-events branch 4 times, most recently from ec05307 to bff523b Compare September 12, 2024 14:19
@matiwinnetou matiwinnetou marked this pull request as ready for review September 12, 2024 14:20

private String version;
private String user;
//private String correlationId; // TODO
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

TODO in future tickets

val txData = blockchainTransactions.serialisedTxData();
val l1SubmissionData = transactionSubmissionService.submitTransaction(txData);
val l1SubmissionData = transactionSubmissionService.submitTransactionWithPossibleConfirmation(txData);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Upon consultation with @satran004 we decided to go for this solution for now but with more clever timeouts.

Ticket to move to a more async, streamlined implementation that allows to submit transactions one by one, one after another without causing UTxO contention issues:
bloxbean/cardano-client-lib#435

@matiwinnetou matiwinnetou merged commit 813182b into main Sep 13, 2024
2 of 3 checks passed
@matiwinnetou matiwinnetou deleted the refakt-events branch September 13, 2024 12:52
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.

2 participants