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

Thea: Fisherman Model #907

Merged
merged 113 commits into from
Feb 20, 2024
Merged

Thea: Fisherman Model #907

merged 113 commits into from
Feb 20, 2024

Conversation

Gauthamastro
Copy link
Member

@Gauthamastro Gauthamastro commented Feb 1, 2024 •
edited
Loading

Describe your changes

  1. Updates Thea to Fisherman model
  2. Removes the Thea aggregator dependency
  3. Automatic claim of Thea deposits
  4. Bug fixes to Thea authority set rotation
  5. On-chain signature aggregation
  6. Ethereum and other EVM chains support

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • I removed all Clippy and Formatting Warnings.
  • I added required Copyrights.

Gauthamastro and others added 30 commits January 5, 2024 07:50
@Gauthamastro
Add on-chain signature aggregation
01e8252
@Gauthamastro
Added Fisherman and dealing with misbehaviour logic
f475a46
@Gauthamastro
Thea core logic completed
302eb2c
@Gauthamastro
Fix runtime
b7b7732
@zktony
Added chnages for eth
1c95c94
@Gauthamastro
Update thea logic
919e0f7
@Gauthamastro
Added changes for Ethereum (#896)
badf4c4 
## Describe your changes
Changed types and added new extrinsic to support Eth.
@Gauthamastro
cargo fmt
9a2682a
@Gauthamastro
fix thea tests
4199ff5
@Gauthamastro
fix thea-executor tests
423c4e9
@Gauthamastro
Update Thea message handler
753e224
@Gauthamastro
Fix tests
f1f2f6c
@Gauthamastro
Fix benchmarks
84ddda8
@Gauthamastro
Resolve clippy errors
d53e1f5
@Gauthamastro
cargo fmt
01eb5ee
@Gauthamastro
Fix benchmarking
d5f37bb
@zktony
Added sign_prehash
6c88298
@zktony
Removed comment
507b15d
@Gauthamastro
Changed sign to sign_prehash (#897)
79abe06 
## Describe your changes
Changed sign to sign_prehash.
@Gauthamastro
fix cargo test
428b623
@zktony
Added benchmarks
8f9921b
@zktony
Added on_init benchmark
8bccd4a
@zktony
Added benchmark code for Thea Pallet and thea Executor Pallet (#905)
7d36fcc 
## Describe your changes
Added benchmark code for Thea Pallet and thea Executor Pallet.
@serhii-temchenko
feat(weights): Updated thea weights
afc2717
@zktony
Resolved all the comments
65ac207
@zktony
Resolved all the comments (#906)
3e9f25c 
## Describe your changes
Resolved all the comments.
@zktony
Fixed ci issues
1c294d5
@zktony
Fix CI Issues
9e6cc7e
@serhii-temchenko
feat(weights): Updated weights for thea related pallets
d9c595e
@zktony
Fixed tests
85764bd
Gauthamastro and others added 10 commits February 12, 2024 14:49
@Gauthamastro
cargo fmt
6199106
@zktony
Fixed tests
4c0718a
@zktony
Fixed formatting
6d90616
@zktony
Merge branch 'zk-uncomp-key' of https://github.com/Polkadex-Substrate…
f8a13a7 
…/Polkadex into zk-uncomp-key
@zktony
Updated param type
39cc1c8
@Gauthamastro
Added support for Uncompressed Key (#910)
fb8c700
@Gauthamastro
Merge branch 'Develop' into gj/exchange-script
5033743
@zktony
Fixed clippy warnings
c46038e
@Gauthamastro
Cargo fmt with stable
2cb552e
@Gauthamastro
Crowdloan Exchange Scripts (#912)
22100bb 
## Describe your changes

1. Added two unsigned extrinsics to initialize and claim rewards by
exchanges

## Issue ticket number and link
Resolves #911
Polkadex-Substrate/issue-tracker#271
## Checklist before requesting a review
- [x] I have performed a self-review of my code.
- [ ] If it is a core feature, I have added thorough tests.
- [x] I removed all Clippy and Formatting Warnings. 
- [ ] I added required Copyrights.
@Gauthamastro Gauthamastro removed the request for review from nuel77 February 20, 2024 05:05
@Gauthamastro Gauthamastro merged commit dc53341 into mainnet-release Feb 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nuel77 nuel77 nuel77 left review comments

@zktony zktony zktony approved these changes

@serhii-temchenko serhii-temchenko serhii-temchenko approved these changes

Assignees

@zktony zktony

Labels
A0-PleaseReview Pull request needs code review.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Gauthamastro @serhii-temchenko @mt-gareth @nuel77 @zktony