martomi / chiadog

A watch dog providing a peace in mind that your Chia farm is running smoothly 24/7.
MIT License
457 stars 121 forks source link

Fix wallet events #119

Closed skweee closed 3 years ago

skweee commented 3 years ago
bhy commented 3 years ago

You are right that the amount is no longer quoted thus the parser needs to be fixed. However I think the new regex should accept both the quoted and non-quoted version for the sake of backward compatibility.

skweee commented 3 years ago

Thanks for your feedback. Initially I also thought about providing backwards compatibility in this case. However, that would mean supporting older chia versions that still have bugs which compromise the overall health of the network (e.g. negative spends). I therefore think it is better to drop backwards compatibility and point people to upgrading their chia installation.

If there is general concensus about the need for backwards compatibility, I'd be happy to include it in this PR though.

bhy commented 3 years ago

@skweee that sounds quite reasonable! Thank you for the work!