quasselgrep icon indicating copy to clipboard operation
quasselgrep copied to clipboard

Missing timestamps #19

Open zkrx opened this issue 4 years ago • 1 comments

This PR fixes issue #19:

  • on sqlite3 >= 31, timestamps now display properly
  • on sqlite3 >= 31, filtering based on timestamps (-t option) now works

For sqlite3 databases, we read schemaversion on connect and store it into the options object. That way, we can deduce the timestamp unit (milliseconds or seconds) from other files. This approach was suggested by @digitalcircuit in #19 [1].

I've just started using quasselgrep, so perhaps I missed something. Let me know if that's the case. Also, I cannot test with databases < 31, but old behavior should be left untouched in that case.

[1] https://github.com/fish-face/quasselgrep/issues/19#issuecomment-482753420

zkrx avatar Dec 31 '20 00:12 zkrx

Hello, anything preventing the merge :)?

zkrx avatar Feb 03 '21 09:02 zkrx