IMAP list/rename commands
ClosedPublic
Actions

Authored by mollekopf on Oct 10 2024, 10:13 AM.

Details

Reviewers

None

Group Reviewers

Restricted Project

Commits

rK54090f602ac5: IMAP list/rename commands

Diff Detail

Repository

rK kolab

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mollekopf requested review of this revision.Oct 10 2024, 10:13 AM

mollekopf created this revision.

Harbormaster completed remote builds in B50745: Diff 14286.Oct 10 2024, 10:13 AM

mollekopf added a reviewer: Restricted Project.Oct 10 2024, 10:34 AM

Looks to me that the folders passed to the command need to be in Cyrus internal format user/{$localpart}/{$mailbox}@{$domain}. That's not very user-friendly. Also tests for the new IMAP methods would be welcome, less important for the command itself.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 20 2024, 11:47 AM

Closed by commit rK54090f602ac5: IMAP list/rename commands. · Explain Why

This revision was automatically updated to reflect the committed changes.

mollekopf added a commit: rK54090f602ac5: IMAP list/rename commands.