Michael Ackley

@demget please rebase and add a test to verify.

Related, the branch for the #464 was merged, see #463 released with version v0.8.1, https://github.com/quickfixgo/quickfix/releases/tag/v0.8.1, please open another issue and link the issue appropriately if it persists

On a re-run it does kick out > filestore.go:225: File is not `gofmt`-ed with `-s` (gofmt) if targetSeqNum, err := strconv.Atoi(strings.Trim(string(targetSeqNumBytes),"\r\n")); err == nil { Give that a quick fix...

@adam-talos thanks 👍

@varunbpatil I believe this is resolved by changes in https://github.com/quickfixgo/quickfix/releases/tag/v0.9.1 and https://github.com/quickfixgo/quickfix/releases/tag/v0.9.2 . Try 0.9.2 and please re-open issue if that is not the case.

Give it a shot again if you still need it, apologies for the delay - `brew tap quickfixgo/qf && brew install qf`

@varunbpatil worth considering, though most of the most popular open source projects seem to target at least 1.20, and even if we don't utilize the most recent language features, I'd...

@Yu-Xie if you would change `SendAppToTarget ` to `SendBatchToTarget` and `SaveMessagesAndIncrNextSenderMsgSeqNum ` to `SaveBatchAndIncrNextSenderMsgSeqNum` I will merge. I'd like to keep func names immediately obvious to prevent accidental misuse. Also...

@mrefky I believe this is resolved in https://github.com/quickfixgo/quickfix/releases/tag/v0.9.1 please re-open issue if that is not the case.

@jbzion Can you provide some example message strings demonstrating what you need? This isnt supported right now, but should be added to order message fields in the order they appear...