Skip to content

Fix send extrinsic function for finalized txs#4803

Merged
mnaamani merged 4 commits intoJoystream:masterfrom
zeeshanakram3:fix_sendExtrinsic_function_for_finalized_txs
Jul 18, 2023
Merged

Fix send extrinsic function for finalized txs#4803
mnaamani merged 4 commits intoJoystream:masterfrom
zeeshanakram3:fix_sendExtrinsic_function_for_finalized_txs

Conversation

@zeeshanakram3
Copy link
Copy Markdown
Contributor

addresses #4802

@vercel
Copy link
Copy Markdown

vercel bot commented Jul 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
pioneer-testnet ⬜️ Ignored (Inspect) Jul 17, 2023 11:15am

@zeeshanakram3 zeeshanakram3 requested a review from mnaamani July 17, 2023 06:07
Copy link
Copy Markdown
Member

@mnaamani mnaamani left a comment

Choose a reason for hiding this comment

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

LGTM.

We experienced this also in running integration tests where a large number of transactions were being generated, and I can see the same solution was applied.

Can you please just do minor version bump for all three packages.

@zeeshanakram3
Copy link
Copy Markdown
Contributor Author

Can you please just do minor version bump for all three packages.

Addressed in 048f23c

@mnaamani mnaamani self-requested a review July 18, 2023 08:29
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