Consider adding last_updated field to PaymentDetails

[tnull]

This would allow the user to query for 'recent' payments based on a suitable filter.

In order to do this nicely we should however consider implementing Writeable/Readable for SystemTime upstream.

[jbesraa]

Can you please say more about the Writeable/Readable for SystemTime ?

If we add last_updated to PaymentDetails and PaymentDetailsUpdate, while in the later its set to last_updated: SystemTime::now() .duration_since(UNIX_EPOCH) .expect("Time went backwards") .as_secs(),
wouldnt that work?

iam actually assuming you are referring to unix time but not sure thats correct?

[tnull]

Yes, we could consider adding just the UNIX timestamp, but might also want to add the Writeable/Readable impl to rust-lightning, we'll see. I will pick this up shortly as part of the BOLT12 work, where I'll touch the PaymentDetails a lot anyways.