Skip to content

Surface ComposeMessageError details to user - #618

Merged
tomholub merged 3 commits into
masterfrom
feature/issue-586-surface-ComposeMessageError-details
Oct 5, 2021
Merged

Surface ComposeMessageError details to user#618
tomholub merged 3 commits into
masterfrom
feature/issue-586-surface-ComposeMessageError-details

Conversation

@ivan-ushakov

@ivan-ushakovivan-ushakov commented Oct 5, 2021

Copy link
Copy Markdown
Contributor

close#586


Tests(delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@tomholubtomholub left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@tomholub
tomholub enabled auto-merge (squash) October 5, 2021 20:32
@tomholub
tomholub merged commit 93c87c2 into masterOct 5, 2021
@tomholub
tomholub deleted the feature/issue-586-surface-ComposeMessageError-details branch October 5, 2021 22:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

surface ComposeMessageError details to user

3 participants

@ivan-ushakov@tomholub@flowcrypt-machine-user