You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
returnt('mail', 'No sent mailbox configured. Please pick one in the account settings.')
},
[ManyRecipientsError.getName()]() {
returnt('mail', 'You are trying to send to many recipients in To and/or Cc. Consider using Bcc to hide recipient addresses.')
},
// eslint-disable-next-line n/handle-callback-err
default(error) {
returnundefined
},
})
this.warning=awaitmatchError(error, {
[AttachmentMissingError.getName()]() {
returnt('mail', 'You mentioned an attachment. Did you forget to add it?')
},
// eslint-disable-next-line n/handle-callback-err
default(error) {
returnundefined
},
})
} finally {
this.sending=false
}
NoSentMailboxConfiguredError and ManyRecipientsError are mapped from the backend exceptions1, but the backend does not throw SentMailboxNotSetException and ManyRecipientsException anymore.
The phpdoc for IMailTransmission.sendMessage is outdated.
NewMessageModal.onSend may put a draft into the outbox without having a draft saved already (1), update a message in the outbox (2), update an existing draft and put it into the outbox (3).
Steps to reproduce
Expected behavior
As a User, I expected a hint/warning when something went wrong.
Actual behavior
Nothing.
To do
Additional info
mail/src/components/NewMessageModal.vue
Lines 416 to 441 in c1d14c4
IMailTransmission.sendMessage
is outdated.NewMessageModal.onSend
may put a draft into the outbox without having a draft saved already (1), update a message in the outbox (2), update an existing draft and put it into the outbox (3).To simulate a failure for case 1 or 3:
Screencast.from.2024-07-11.13-24-49.webm
There's no feedback that something went wrong, I'm just taking back to the composer, and it says, "Draft saved".
Footnotes
https://github.com/nextcloud/mail/blob/c1d14c4fa32f8d66543db9dc2402d5914127d7d8/src/errors/convert.js#L15-L24 ↩
The text was updated successfully, but these errors were encountered: