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

substrate module @ pba3 #749

Merged
merged 12 commits into from
Jul 24, 2023
Merged

substrate module @ pba3 #749

merged 12 commits into from
Jul 24, 2023

Conversation

kianenigma
Copy link
Member

closes #620 and in general preps substrate module for pba3.

})
let (signer, tip, nonce) = Self::solution_validate_transaction_inner(&ext)?;
let provides = vec![(signer.clone(), nonce).encode()];
let requires = vec![(signer.clone(), nonce.saturating_sub(1)).encode()];
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is slightly wrong, probably because I explained it too hastily. Please verify for yourself.

The requires tags should include all the transactions and only those transactions that we are still waiting to see at the time of validation. So if you find that the stored nonce is already one lower than the current tx, then add no requires tags.

But if you find that the stored nonce is three below the current tx, then you need two requires tags. (for nonce - 1 and nonce - 2).

@@ -1,80 +1,8 @@
# Substrate Frameless Node Template
# Framelss Activity V3

Choose a reason for hiding this comment

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

#718 not gonna happen in this PR I guess? We can do latter, I see https://github.com/Polkadot-Blockchain-Academy/PBA3-frameless-node-template perhaps we can rm this code then?

Comment on lines +1 to +5
rustup target add wasm32-wasi
rustup target add wasm32-unknown-unknown

cargo init --lib
apply stuff (`cdylib`, `#[no_mangle]`)

Choose a reason for hiding this comment

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

is this file just internal facing scratch pad for you? if needed to run the module or needed to be shared with students, a touch of polish here to define how to use it for other instructors is needed ( can be a follow up PR)

Copy link

@nuke-web3 nuke-web3 left a comment

Choose a reason for hiding this comment

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

nice work 🦸

@nuke-web3 nuke-web3 merged commit 1b834cb into main Jul 24, 2023
@nuke-web3 nuke-web3 deleted the kiz-substrate-pba3-second branch July 24, 2023 16:02
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.

Tracking Issue Substrate Module PBA3
3 participants