Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AllAccounts: AllFolders.listMessages(): must be a Collection #325

Open
jermy-c opened this issue Dec 19, 2024 · 2 comments
Open

AllAccounts: AllFolders.listMessages(): must be a Collection #325

jermy-c opened this issue Dec 19, 2024 · 2 comments
Assignees

Comments

@jermy-c
Copy link
Collaborator

jermy-c commented Dec 19, 2024

Reproduction

  1. Select All accounts
  2. Select a folder in all accounts
  3. Might need to click through the folders

Actual Result

  1. Yellow banner error: must be a Collection

Expected Result

  1. No error

Stack

error.ts:6 Error: must be a Collection
    at assert (svelte-collections.js?v=790a80fa:82:11)
    at AdditionCollection.addColl (svelte-collections.js?v=790a80fa:1153:5)
    at mergeColl (svelte-collections.js?v=790a80fa:1125:11)
    at AllFolders.listMessages (AllFolders.ts:36:12)
    at async loadFolder (MailApp.svelte:40:28)
@benbucksch benbucksch changed the title must be a Collection AllFolders.listMessages: must be a Collection Dec 28, 2024
@benbucksch
Copy link
Collaborator

benbucksch commented Jan 8, 2025

I've disabled this feature for now, due to too many bugs. Before testing this, revert the one-liner git commit e30bde2 to re-enable the feature.

@benbucksch benbucksch assigned nix-universe and unassigned jermy-c Jan 8, 2025
@benbucksch
Copy link
Collaborator

@benbucksch benbucksch changed the title AllFolders.listMessages: must be a Collection AllAccounts: AllFolders.listMessages(): must be a Collection Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants