Skip to content

submit iris milestone 1#327

Merged
semuelle merged 2 commits intow3f:masterfrom
ideal-lab5:iris_milestone_1
Dec 24, 2021
Merged

submit iris milestone 1#327
semuelle merged 2 commits intow3f:masterfrom
ideal-lab5:iris_milestone_1

Conversation

@driemworks
Copy link
Copy Markdown
Contributor

Milestone Delivery Checklist

Link to the application pull request: w3f/Grants-Program#657

@semuelle
Copy link
Copy Markdown
Contributor

Thank you for the delivery, Tony. We will look into it as soon as possible.

@semuelle
Copy link
Copy Markdown
Contributor

Hey Tony. Sorry for the late reply, but I am now looking at your delivery. Everything looks great so far (I especially appreciate the thorough documentation), but I am unable to compile the pallet. This is on iris_milestone_1. Can you verify that this is compiling on your end?

error[E0599]: no function or associated item named `pre_dispatch` found for struct `pallet::Pallet` in the current scope
   --> bin/node-template/pallets/template/src/mock.rs:22:1
    |
22  | / construct_runtime!(
23  | |     pub enum Test where
24  | |         Block = Block,
25  | |         NodeBlock = Block,
...   |
32  | |     }
33  | | );
    | |_^ function or associated item not found in `pallet::Pallet<mock::Test>`
    |
   ::: bin/node-template/pallets/template/src/lib.rs:140:5
    |
140 |       pub struct Pallet<T>(_);
    |       ------------------------ function or associated item `pre_dispatch` not found for this
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following traits define an item `pre_dispatch`, perhaps you need to implement one of them:
            candidate #1: `SignedExtension`
            candidate #2: `ValidateUnsigned`
    = note: this error originates in the macro `construct_runtime` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0599]: no function or associated item named `validate_unsigned` found for struct `pallet::Pallet` in the current scope
   --> bin/node-template/pallets/template/src/mock.rs:22:1
    |
22  | / construct_runtime!(
23  | |     pub enum Test where
24  | |         Block = Block,
25  | |         NodeBlock = Block,
...   |
32  | |     }
33  | | );
    | |_^ function or associated item not found in `pallet::Pallet<mock::Test>`
    |
   ::: bin/node-template/pallets/template/src/lib.rs:140:5
    |
140 |       pub struct Pallet<T>(_);
    |       ------------------------ function or associated item `validate_unsigned` not found for this
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following traits define an item `validate_unsigned`, perhaps you need to implement one of them:
            candidate #1: `SignedExtension`
            candidate #2: `ValidateUnsigned`
    = note: this error originates in the macro `construct_runtime` (in Nightly builds, run with -Z macro-backtrace for more info)

@semuelle semuelle self-assigned this Dec 23, 2021
@driemworks
Copy link
Copy Markdown
Contributor Author

My apologies, looks like I missed a commit when I created the fork.

The issue is the inclusion of 'ValidateUnsigned' here.
https://github.com/iridium-labs/substrate/blob/eebe9208e84bffc468b2eb4c50e6a86c6f0405d4/bin/node-template/pallets/template/src/mock.rs#L31

I've added an additional commit to the iris_milestone_1 branch that removed 'ValidateUnsigned' as well as enhances the test suite by adding support for mocking the offchain ipfs behavior as well as more fully testing the OCW functionality.
commit hashes: ed3e84609a12f88b99a1d7d331adccad7017c7d6 and 86819ba749a082ee79c7eacf67038e451a4bf4b3

@semuelle
Copy link
Copy Markdown
Contributor

semuelle commented Dec 23, 2021

I can't find these commits on the repo, @driemworks.

Thanks for the update. I was able to compile everything, run the tests and follow the walkthrough. Great work on the documentation, thanks for that. Your milestone is hereby accepted. Evaluation notes here.

Could you resubmit your invoice with the payment address included? I will then forward it for processing.

@semuelle semuelle merged commit ef86688 into w3f:master Dec 24, 2021
@driemworks
Copy link
Copy Markdown
Contributor Author

Thank you Sebastian! I've resubmitted the invoice with the payment address included.

@Noc2
Copy link
Copy Markdown
Contributor

Noc2 commented Jan 8, 2022

@driemworks we received your email to support@polkadot.network. Thanks for chasing us. I shared your latest invoice with the invoice team. Sorry for the delay here. You should receive your payment in the next two weeks.

@RouvenP
Copy link
Copy Markdown

RouvenP commented Jan 11, 2022

hi @driemworks, we transferred the payment today. Thank you & apologies for the delay.

@driemworks driemworks mentioned this pull request Mar 2, 2022
5 tasks
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