tendermint / spn

A blockchain to launch blockchains.
Other
112 stars 43 forks source link

refactor(`launch`): merge request related messages into `MsgSendRequest` #947

Closed lumtis closed 2 years ago

lumtis commented 2 years ago

All requests merged into a single MsgSendRequest message Same functionality, 3700 lines removed

To test: use the spnd CLI, interface remains the same

codecov[bot] commented 2 years ago

Codecov Report

Merging #947 (a9203b2) into develop (c65411c) will decrease coverage by 0.07%. The diff coverage is 58.90%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #947      +/-   ##
===========================================
- Coverage    10.75%   10.67%   -0.08%     
===========================================
  Files          280      272       -8     
  Lines        67161    64942    -2219     
===========================================
- Hits          7220     6931     -289     
+ Misses       59780    57849    -1931     
- Partials       161      162       +1     
Impacted Files Coverage Δ
x/launch/simulation/simulation.go 0.00% <0.00%> (ø)
x/launch/types/tx.pb.go 0.53% <ø> (+0.04%) :arrow_up:
x/launch/types/msg_send_request.go 30.76% <30.76%> (ø)
x/launch/keeper/msg_send_request.go 100.00% <100.00%> (ø)
x/launch/keeper/request.go 100.00% <100.00%> (ø)
x/launch/types/request_content.go 100.00% <100.00%> (ø)
x/campaign/simulation/store.go 82.43% <0.00%> (-2.03%) :arrow_down: