dzmitry-lahoda

@bluele ``` function sendPacket(Lib.Packet memory packet) internal virtual override(AppBase, HookMiddlewarePacketSender, FeeMiddlewarePacketSender) { // fee -> hook -> app -> base super.sendPacket(packet); } ``` looks wrong. app should call sendPacket of...

on receive ` fee -> hook -> app -> base` correct, or send `base -> app -> hook -> fee` is correct

``` When nesting an application, the module must make sure that it is in the middle of communication between core IBC and the application in both directions. Developers should do...

WOW. I did know about override feature. trying.

2 issue with approach: - need to make all packet data to be memory - that leads to next via-ir issue - also need to override ALL methods in middleware...

@bluele so in this https://github.com/hyperledger-labs/yui-ibc-solidity/pull/220 1. needed to split app base and app base with defaults, so middlewares do no need to specify ALL dummy methods 2. via-ir issue. if...

``` coordinator.go:501: ack found: {"result":"AQ=="} coordinator.go:510: delay for ack@3018 40.852138202s === RUN TestChainTestSuite/TestTimeoutAndClose suite.go:87: test panicked: Account not found goroutine 98 [running]: runtime/debug.Stack() /nix/store/23wrmhyfnwjxc9cvy3lab9xj551ic53c-go-1.20.8/share/go/src/runtime/debug/stack.go:24 +0x65 github.com/stretchr/testify/suite.failOnPanic(0xc00091c820, {0x1251420, 0xc00071a9c0}) /home/dz/go/pkg/mod/github.com/stretchr/[email protected]/suite/suite.go:87 +0x3b...

``` For more information, try '--help'. bash-5.2$ forge build [⠰] Compiling... [⠑] Compiling 85 files with 0.8.21 [⠢] Solc 0.8.21 finished in 5.29s Error: Compiler run failed: Error: Compiler error...

fixed via ir. unit tests passed. i will test middleware on REAL input from Centauri and Osmosis cosmos chain soon. will add that copy into unit test after

for me it is best form of documentation, sometimes i miss put some attributes and they do not work, but having schema allow to be 100% i do rigth.