Skip to content

[PB-1974]: Feature/draft emails#68

Merged
xabg2 merged 9 commits into
masterfrom
feature/draft-emails
Jun 23, 2026
Merged

[PB-1974]: Feature/draft emails#68
xabg2 merged 9 commits into
masterfrom
feature/draft-emails

Conversation

@xabg2

@xabg2 xabg2 commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Adding new EPs to save a Draft, update a Draft, Send a Draft and discard a Draft.

@xabg2 xabg2 requested a review from jzunigax2 June 19, 2026 15:17
@xabg2 xabg2 self-assigned this Jun 19, 2026
@xabg2 xabg2 added the enhancement New feature or request label Jun 19, 2026
threading?: ThreadingHeaders,
): Promise<{ id: string }> {
if (dto.draftId) {
console.log('THERE IS A DRAFT ID');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this debug log

this.resolveMailboxId(userEmail, 'sent'),
]);

console.log('SUBMIT EXISTING DRAFT', draftId);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another debug log

Comment on lines +588 to +590
if (!existingDraft) {
throw new Error('Draft not found');
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this isn't catched anywhere, meaning this gets surfaced to the client as a 500 when it should be a 404. I dont believe this file should import nest's not found error and throw from here though

@xabg2 xabg2 requested a review from jzunigax2 June 22, 2026 08:40
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
69.1% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@xabg2 xabg2 merged commit cbf256a into master Jun 23, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants