fix-timesmart appears to fix the issue I experienced with incorrect transaction timestamps when importing watch-only addresses. After compiling, I used -rescan as well as -zapwallettxes, neither of which resolved the issue (on my end). After making a new wallet and reimporting with importmulti, the timestamps are correct.

Unsure on the reasoning behind this initial behaviour so can't ACK/NACK until Luke or someone else explains. But looks like src/wallet/test/wallet_tests.cpp needs updating too.

Note I don't think GitHub doesn't understand "Fixes X and Y" when it does it's auto-close, you probably have to write Fixes X, fixes Y.

@luke-jr

The idea IIRC was simply to ensure timestamps move forward at least in ordinary circumstances.

There hasn't been much activity lately and the patch still needs rebase, so I am closing this for now. Please let me know when you want to continue working on this, so the pull request can be re-opened.