fix persist add store transaction fix #447

quickfixgo / quickfix

The Go FIX Protocol Library :rocket:

https://www.quickfixgo.org/

Other

735 stars 288 forks source link

fix persist add store transaction fix #447 #525

Closed sami-sweng closed 1 year ago

ackleymi commented 1 year ago

I like this change a lot so far, I'm just hesitant to merge it without a unit test proving that it solves #447 , the existing store_test.go should be a good place to start

ackleymi commented 1 year ago

Merging into a wip branch for now, want to test this for myself, will reopen PR into main after

ackleymi commented 1 year ago

https://github.com/quickfixgo/quickfix/pull/528