Add LAST_MODIFIED attribute when exporting

[ixzhao]

No description provided.

[sissbruecker]

What would be the benefit of adding this?

[ixzhao]

What would be the benefit of adding this?

Trying to reserve all original info.
When I export or import bookmarks in my browser or linkding, last modified info gets lost.

[sissbruecker]

Preserving information is not really a benefit in itself, unless it supports some other feature. The modified date isn't really used anywhere in the app, which is why I'm asking.

[ixzhao]

Preserving information is not really a benefit in itself, unless it supports some other feature. The modified date isn't really used anywhere in the app, which is why I'm asking.

As far as I'm concerned, it's helpful to me because I don't want to lose such infomation when I migrate bookmarks between browser and linkding. Besides, if someone need to sort by modified date in linkding, that would be helpful too, although one can only sort by added date and title at the moment.

[ixzhao]

@sissbruecker
Would you consider this, or should I close the PR?

[sissbruecker]

I saw that LAST_MODIFIED is something that's also supported by browsers, with that I'm fine with adding it.

One thing that's missing from this PR is tests, would be good if you could add those.

[ixzhao]

I saw that LAST_MODIFIED is something that's also supported by browsers, with that I'm fine with adding it.

Exactly that is supported by firefox and edge which I'm using, And that's why I made this PR.

One thing that's missing from this PR is tests, would be good if you could add those.

I try hard to add tests. When doing that, I find it's hard to split into two PR, and so, I merge "parse-LAST_MODIFIED" PR into this one. Please review.

[sissbruecker]

Thanks!