Skip to content

feat: added post_block_update prevalidation to extrinsics disptach#1448

Open
Klapeyron wants to merge 1 commit into
mainfrom
feat/post_block_update_prevalidation
Open

feat: added post_block_update prevalidation to extrinsics disptach#1448
Klapeyron wants to merge 1 commit into
mainfrom
feat/post_block_update_prevalidation

Conversation

@Klapeyron
Copy link
Copy Markdown
Contributor

Overview

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • All commits are signed off (git commit -s) for the DCO
  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@ozgb
Copy link
Copy Markdown
Contributor

ozgb commented May 1, 2026

Incompatible with this PR: #1443
post_block_update un-persists the input state in 1443

1443 is not yet merged, so we're good for now

@Klapeyron Klapeyron force-pushed the feat/post_block_update_prevalidation branch from 52921ef to 666d5fb Compare May 4, 2026 07:00
@Klapeyron
Copy link
Copy Markdown
Contributor Author

@ozgb thank you for the notice! I made quick rebase on top of ozgb-ledger-intermediate-states and it rebased fluently, do you prefer to land your changes first? Mine are shorter, I can postpone them until you deliver #1443

@Klapeyron Klapeyron marked this pull request as ready for review May 4, 2026 07:04
@Klapeyron Klapeyron requested a review from a team as a code owner May 4, 2026 07:04
Comment thread ledger/src/versions/common/api/ledger.rs Outdated
@Klapeyron Klapeyron force-pushed the feat/post_block_update_prevalidation branch 2 times, most recently from 693ff09 to e070d94 Compare May 4, 2026 09:19
Signed-off-by: Tomasz Bartos <tomasz.bartos@shielded.io>
@Klapeyron Klapeyron force-pushed the feat/post_block_update_prevalidation branch from e070d94 to 23cb62c Compare May 4, 2026 09:21
@m2ux
Copy link
Copy Markdown
Contributor

m2ux commented May 6, 2026

Does this PR now replace #1438?

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.

4 participants