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.

Sign up for GitHub

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

Jump to bottom

I94 storage revamp merkle #101

Merged
merged 44 commits into from
Feb 16, 2023
Merged

I94 storage revamp merkle #101

merged 44 commits into from
Feb 16, 2023

Conversation

PopcornPaws
Copy link
Contributor

@PopcornPaws PopcornPaws commented Feb 15, 2023 •
edited
Loading

Description

Aims to close #94 via tons of changes...

  • requirement filters: allowlists and child roles (you need to have another role to enter) are checked on-chain
  • allowlists are merkelized and only the merkle root is stored on-chain (the allowlist gets written to the off-chain storage)
  • split transactions into multiple, smaller transactions (easier to benchmark)
  • 1600+ lines of unit tests for the guild pallet
  • updated wasm bindings

@PopcornPaws
Identity revamp wip.
1aae3bb
@PopcornPaws
Updated identities in common.
0afbc49
@PopcornPaws
Updated identity handling in pallet. Tests + oracle remaining.
dd0776d
@PopcornPaws
Fixed guild pallet tests.
f5bb991
@PopcornPaws
Runtime builds as well.
f7fcf09
@PopcornPaws
WIP integration tests.
396584d
@PopcornPaws
Integration tests pass with mock oracle.
264f47d
@PopcornPaws
Updated oracle code.
367a2ea
@PopcornPaws
Only the token test remaining.
10cb8af
@PopcornPaws
All integration tests pass.
672ec5f
@PopcornPaws
Almost done...
56b8601
@PopcornPaws
Nicer user data storage.
c11d642
@PopcornPaws
All tests pass.
ef14157
@PopcornPaws
Started refactoring requirements for allowlist.
28d469a
@PopcornPaws
Generic Filter to OnChainFilter conversion.
f2142dd
@PopcornPaws
on-chain and off-chain requirements are better separated.
7e58579
@PopcornPaws
Engine cleanup.
aa53aab
@PopcornPaws
More engine cleanups.
3af5296
@PopcornPaws
Guild pallet wip.
f5ae1e2
@PopcornPaws
On chain allowlist merkle proof checking.
0749b88
@PopcornPaws
Wip guild pallet, only testing remains.
ab0fb28
@PopcornPaws
Allowlist creation and off-chain storage tested.
20968d8
@PopcornPaws
Guild and role creation tested.
2f78006
@PopcornPaws
Identity registration tests cleaned up.
dcab888
@PopcornPaws
WIP final pallet tests.
e7b8afb
@PopcornPaws
Pallet thoroughly tested.
b44345a
@PopcornPaws
Small upgrades.
95c830e
@PopcornPaws
WIP client cleanup.
39336f3
@PopcornPaws
Queries fixed.
5d38c1c
@PopcornPaws
Transactions cleaned up.
3d7cf4b
@PopcornPaws PopcornPaws self-assigned this Feb 15, 2023
@PopcornPaws
Copy link
Contributor Author

PopcornPaws commented Feb 16, 2023 •
edited
Loading

What's left:

  • fix the secp256k1 build errors (that only occur in the ci/cd test pipeline...probably some missing clang dependencies are needed to be installed)
  • discuss whether the frontend needs additional wasm bindings

@PopcornPaws PopcornPaws marked this pull request as ready for review February 16, 2023 14:26
@PopcornPaws PopcornPaws merged commit 8d1433d into main Feb 16, 2023
@PopcornPaws PopcornPaws deleted the I94-storage-revamp-merkle branch February 16, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@PopcornPaws PopcornPaws

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Storage redesign
1 participant
@PopcornPaws