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

fee middleware: grpc queries #488

Merged
merged 55 commits into from
Nov 10, 2021
Merged
Changes from 1 commit
Commits
Show all changes
55 commits
Select commit Hold shift + click to select a range
435efc7
fix: adding second endpoint for async pay fee + renaming types
seantking Oct 1, 2021
8f2c51a
feat: adding escrow logic
seantking Oct 1, 2021
a410af6
feat: updating proto types & escrow logic
seantking Oct 4, 2021
0e6823f
fix: stub fn & proto comment
seantking Oct 5, 2021
f41e520
feat: adding PayFee & PayFeeTimeout & escrow_test
seantking Oct 6, 2021
e4bd49a
test: adding happy path for EscrowPacketFee
seantking Oct 6, 2021
ca7f4e2
fix: comments, error handling
seantking Oct 7, 2021
6a5add7
fix: comments & grammar
seantking Oct 7, 2021
952a3e3
grpc scaffold
charleenfei Oct 8, 2021
535e2c6
test: adding unhappy path for escrow
seantking Oct 8, 2021
5bd0e32
tests(escrow): adding hasBalance check for module acc
seantking Oct 8, 2021
31fcee3
test(PayFee): adding happy path for PayFee tests
seantking Oct 8, 2021
487f78d
tests(PayFee, PayFeeTimeout): adding tests
seantking Oct 8, 2021
f25932f
fix: adding relayers back to IdentifiedPacket
seantking Oct 11, 2021
90284c7
wip
charleenfei Oct 11, 2021
ea37edc
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 11, 2021
38566bc
fix: removing refund acc from key
seantking Oct 11, 2021
ceb49ca
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 11, 2021
dd0044b
fix: storing IdentifiedPacketFee in state instead of Fee
seantking Oct 11, 2021
526a13d
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 11, 2021
5b4a395
feat: adding msg_server test for registerCPAddr, wiring for codec + s…
seantking Oct 11, 2021
3c2dd24
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 11, 2021
c9af582
test: adding msg_server test for PayPacketFee
seantking Oct 12, 2021
980d748
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 12, 2021
d325146
test: adding PayPacketFeeAsync msg_server test
seantking Oct 12, 2021
b6ef099
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 12, 2021
ea9b30b
chore: updating PayFee -> DistributeFee & minor nits
seantking Oct 13, 2021
ac7584f
nit: removing unnecessary nil check
seantking Oct 13, 2021
5bd3382
update protos
charleenfei Oct 13, 2021
be4b865
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 13, 2021
ec2f0d4
query function
charleenfei Oct 13, 2021
46fe890
prefix & query heights
charleenfei Oct 13, 2021
c44a81c
refactor: add portId to store key & use packetId as param
seantking Oct 13, 2021
a8c3cff
fix: add DeleteFeeInEscrow & remove fee on successful distribution
seantking Oct 13, 2021
4f490ea
tests: adding validation & signer tests for PayFee/Async & updating p…
seantking Oct 13, 2021
f1fde1d
chore: adding NewIdentifiedPacketFee fn
seantking Oct 13, 2021
eeb4376
fix: getter/setter for counterparty address + fix NewIdentifiedPacketFee
seantking Oct 14, 2021
4612bf2
fix: updating EscrowPacketFee with correct usage of coins api
seantking Oct 14, 2021
3494a0f
test: adding balance check for refund acc after escrow
seantking Oct 14, 2021
f4c37e6
fix: remove unncessary errors
seantking Oct 14, 2021
f547a2c
Merge branch 'sean/issue#-259-escrow-logic' of github.com:cosmos/ibc-…
charleenfei Oct 14, 2021
b666224
wip
charleenfei Oct 15, 2021
96999c4
fix tests
charleenfei Oct 27, 2021
0b733ab
test: updating escrow tests + miscellaneous fixes
seantking Nov 3, 2021
2be4ab5
nit: updating var names
seantking Nov 3, 2021
cb2cbc3
docs: godoc
seantking Nov 3, 2021
66454c6
refactor: IdentifiedPacketFee & Fee no longer pointers
seantking Nov 3, 2021
06d4c25
fixes: small fixes
seantking Nov 3, 2021
8757f9a
fix merge conflict
charleenfei Nov 3, 2021
f7acaa4
Merge branch 'ics29-fee-middleware' into issue#479-grpc-query
charleenfei Nov 8, 2021
23c98ab
initial fix
charleenfei Nov 8, 2021
0864c55
interim fix
charleenfei Nov 8, 2021
8779ed7
merge conflict changes
charleenfei Nov 8, 2021
22b62b5
updates for nits
charleenfei Nov 8, 2021
fc4db7a
update for nits
charleenfei Nov 10, 2021
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
Prev Previous commit
Next Next commit
fix: adding relayers back to IdentifiedPacket
  • Loading branch information
seantking committed Oct 11, 2021
commit f25932f1cad76dc7c8eb1295cd570292ecd7e115
1 change: 1 addition & 0 deletions docs/ibc/proto-docs.md
Original file line number Diff line number Diff line change
@@ -649,6 +649,7 @@ Fee associated with a packet_id
| ----- | ---- | ----- | ----------- |
| `packet_id` | [ibc.core.channel.v1.PacketId](#ibc.core.channel.v1.PacketId) | | |
| `fee` | [Fee](#ibc.applications.fee.v1.Fee) | | |
| `relayers` | [string](#string) | repeated | |



108 changes: 82 additions & 26 deletions modules/apps/29-fee/types/fee.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions proto/ibc/applications/fee/v1/fee.proto
Original file line number Diff line number Diff line change
@@ -19,4 +19,5 @@ message Fee {
message IdentifiedPacketFee {
ibc.core.channel.v1.PacketId packet_id = 1 [(gogoproto.moretags) = "yaml:\"packet_id\""];
Fee fee = 2;
repeated string relayers = 3;
}