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

feat: Display system message when message was partially sent [FS-1570] #14673

Merged
merged 15 commits into from
Feb 22, 2023

Conversation

atomrc
Copy link
Contributor

@atomrc atomrc commented Feb 13, 2023

Sub-taskFS-1570 [web] Show system message indicating who will receive the message later

First batch of handling sending errors when one federated backend is offline.

image

image

image

@codecov
Copy link

codecov bot commented Feb 13, 2023

Codecov Report

Merging #14673 (c654c6a) into dev (9fd1886) will increase coverage by 0.07%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##              dev   #14673      +/-   ##
==========================================
+ Coverage   42.76%   42.83%   +0.07%     
==========================================
  Files         618      621       +3     
  Lines       21127    21226      +99     
  Branches     4842     4876      +34     
==========================================
+ Hits         9034     9092      +58     
- Misses      10958    10979      +21     
- Partials     1135     1155      +20     

@atomrc atomrc force-pushed the feat/display_failed_to_send branch 3 times, most recently from 84d16fa to dc9a14b Compare February 20, 2023 09:51
@atomrc atomrc changed the title feat: Display system message when message was partially sent feat: Display system message when message was partially sent [FS-1570] Feb 21, 2023
@atomrc atomrc marked this pull request as ready for review February 22, 2023 13:05
@atomrc atomrc requested review from a team and otto-the-bot as code owners February 22, 2023 13:05
Copy link
Contributor

@PatrykBuniX PatrykBuniX left a comment

Choose a reason for hiding this comment

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

Nice 👍

atomrc and others added 3 commits February 22, 2023 14:57
…edToSendWarning/FailedToSendWarning.styles.ts

Co-authored-by: Virgile <78490891+V-Gira@users.noreply.github.com>
@atomrc atomrc merged commit bd26e70 into dev Feb 22, 2023
@atomrc atomrc deleted the feat/display_failed_to_send branch February 22, 2023 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants