Page MenuHomePhorge

IMAP data migrator and other improvements
ClosedPublic

Authored by machniak on Jul 7 2024, 10:11 AM.
Tags
None
Referenced Files
F15414048: D4845.id13881.diff
Thu, Sep 19, 6:26 PM
F15398265: D4845.id13878.diff
Thu, Sep 19, 2:20 AM
Unknown Object (File)
Sat, Aug 31, 2:54 PM
Unknown Object (File)
Thu, Aug 29, 8:59 AM
Unknown Object (File)
Wed, Aug 28, 11:20 PM
Unknown Object (File)
Wed, Aug 28, 12:46 PM
Unknown Object (File)
Sat, Aug 24, 5:17 PM
Unknown Object (File)
Aug 17 2024, 2:58 PM
Subscribers
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK21d3bcab8719: IMAP data migrator and other improvements
Summary
  • IMAP data migration
  • DAV can be used as an exporter (but this wasn't really tested yet)
  • Async jobs can handle multiple items at once
  • data:migrator --sync option

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak created this revision.

Looks good to me.

IMO, by going through the import/export interfaces and using the Item abstraction (which I don't think is really necessary) the code becomes unnecessarily rigid, and I liked that about my previous approach better, but this is more consistent with your previous code, so that's positive. In any case, that's just to provide an opinion, this approach should work fine and it's mostly a matter of taste I suppose.

This seems good enough to merge, so let's.

This revision is now accepted and ready to land.Jul 7 2024, 1:33 PM
This revision was automatically updated to reflect the committed changes.