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

IPFS Utilities milestone 2 #838

Merged
merged 4 commits into from
Apr 21, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
80 changes: 80 additions & 0 deletions evaluations/ipfs_utilities_2_randombishop.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,80 @@
# Evaluation



- **Status:** ACCEPTED
- **Application Document:** [IPFS Utilities](https://github.com/w3f/Grants-Program/blob/master/applications/ipfs_utilities.md)
- **Milestone:** 2
- **Kusama Identity:** randombishop (https://sub.id/5Gnixfp6vnznRkr91JgwkxYnCJCyHr8EaBzYfFsUKcTMzVYF)
- **Previously successfully merged evaluation:** [Milestone 1](https://github.com/w3f/Grant-Milestone-Delivery/blob/master/evaluations/ipfs_utilities_1_randombishop.md)

| Number | Deliverable | Accepted | Link | Evaluation Notes |
|-------:|---------------------------------|----------|------------------------------------------------------------------------------------------|-------------------------------|
| 0a. | License | Yes | [Licence](https://github.com/TDSoftware/substrate-ipfs/blob/milestone-1/LICENSE-APACHE2) | Apache 2.0 |
| 0b. | Documentation | Yes | [Doc](https://github.com/TDSoftware/substrate-ipfs/blob/master/docs/IPFS.md) | OK. Improved from milestone 1 |
| 0c. | Testing | Yes | [Doc](https://github.com/TDSoftware/substrate-ipfs/blob/milestone-2/docs/IPFS.md) | OK |
| 0d. | Code | Yes | [Milestone 2](https://github.com/TDSoftware/substrate-ipfs/tree/milestone-2) | OK |
| 0e. | Article | Yes | https://medium.com/p/6cfcb537f660 | OK |
| 1. | Fetch File Implementation | Yes | [Node](https://github.com/TDSoftware/substrate-ipfs/tree/milestone-2) | OK |
| 2. | Web3 Client Extension | Yes | [Example](https://github.com/TDSoftware/substrate-ipfs-example-ui) | OK |
| 3. | Local Node Fetch Implementation | Yes | [Node](https://github.com/TDSoftware/substrate-ipfs/tree/milestone-2) | OK |


## Repositories
(Apache 2.0 licensed)

| Component | Repo | Language |
|---------------:|---------------------------------------------------------|----------|
| Substrate node | https://github.com/TDSoftware/substrate-ipfs | Rust |
| Frontend | https://github.com/TDSoftware/substrate-ipfs-ui | JS |
| Demo example | https://github.com/TDSoftware/substrate-ipfs-example-ui | JS |



## Logs

* cargo test --package pallet-tds-ipfs-core
```
running 19 tests
test tests::tests::test_cid_data ... ok
test mock::__construct_runtime_integrity_test::runtime_integrity_tests ... ok
test tests::tests::test_add_bytes ... ok
test tests::tests::test_cat_bytes ... ok
test tests::tests::test_disconnect ... ok
test tests::tests::test_connect ... ok
test tests::tests::test_generate_id ... ok
test tests::tests::test_addresses_to_utf8_safe_bytes ... ok
test tests::tests::test_offchain_data ... ok
test tests::tests::test_ipfs_command_type_equality ... ok
test tests::tests::test_offchain_data_key ... ok
test tests::tests::test_offchain_storage_data_for_block_number ... ok
test tests::tests::test_offchain_storage_data_for_key ... ok
test tests::tests::test_multiple_bytes_to_utf8_safe_bytes ... ok
test tests::tests::test_store_cid_data_for_block_number ... ok
test tests::tests::test_store_cid_data_for_values ... ok
test tests::tests::test_store_cid_data_for_key ... ok
test tests::tests::test_store_cid_data_for_key_and_clear_afterwards ... ok
test tests::tests::test_store_offchain_data_read_write ... ok

test result: ok. 19 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.01s

```

* cargo test --package pallet-tds-ipfs
```
running 4 tests
test pallet::test_find_value_of_type_in_vector ... ok
test pallet::test_type_equality ... ok
test pallet::test_contains_value_of_type_in_vector ... ok
test mock::__construct_runtime_integrity_test::runtime_integrity_tests ... ok

test result: ok. 4 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
```


## General notes

Clean delivery and nice introduction article. Good-to-have IPFS feature in substrate and works as expected.