guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

Move To Folder: create new #422

New Issue
Closed
opened 2026-06-04 17:24:52 +00:00 by guettli · 1 comment
guettli commented 2026-06-04 17:24:52 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

When I choose "move to folder" from single Mail View, I want to be able to create a new folder.

When I choose "move to folder" from single Mail View, I want to be able to create a new folder.
guettlibot commented 2026-06-04 17:35:03 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

PR opened at https://codeberg.org/guettli/sharedinbox/pulls/423.

Here's what I did:

Implementation summary:

  1. lib/core/repositories/mailbox_repository.dart — added createMailbox(accountId, name) method to the interface (no special role, for user-created folders).

  2. lib/data/repositories/mailbox_repository_impl.dart — implemented createMailbox by routing to the existing IMAP/JMAP private helpers, making the role parameter nullable (String?). The JMAP request now conditionally includes the role field only when non-null (avoids sending an empty/null role to the server).

  3. lib/ui/screens/email_detail_screen.dart — added:

    • _promptNewFolderName() — shows an AlertDialog with a text field + Cancel/Create buttons, properly disposes the controller
    • A "Create new folder…" ListTile at the bottom of the "Move to…" bottom sheet using a sentinel value to distinguish it from folder paths; on selection, prompts for a name, calls createMailbox, then moves the email to the new folder
PR opened at https://codeberg.org/guettli/sharedinbox/pulls/423. Here's what I did: **Implementation summary:** 1. **`lib/core/repositories/mailbox_repository.dart`** — added `createMailbox(accountId, name)` method to the interface (no special role, for user-created folders). 2. **`lib/data/repositories/mailbox_repository_impl.dart`** — implemented `createMailbox` by routing to the existing IMAP/JMAP private helpers, making the `role` parameter nullable (`String?`). The JMAP request now conditionally includes the `role` field only when non-null (avoids sending an empty/null role to the server). 3. **`lib/ui/screens/email_detail_screen.dart`** — added: - `_promptNewFolderName()` — shows an `AlertDialog` with a text field + Cancel/Create buttons, properly disposes the controller - A "Create new folder…" `ListTile` at the bottom of the "Move to…" bottom sheet using a sentinel value to distinguish it from folder paths; on selection, prompts for a name, calls `createMailbox`, then moves the email to the new folder
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels loop/code-done
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#422