Skip to content

Commit

Permalink
NotifTroubleshootingScreen: Make email-sent log line actually reach S…
Browse files Browse the repository at this point in the history
…entry

An alternative approach to zulip#5720, to help us detect potential cases
where the user thought they'd contacted support through the app, but
we don't see an email on our side. (We're not aware of such cases
yet, but in principle it could happen.)

It seems probably fine to do it at the "warn" level rather than
"info". If someone sends one of these emails, it usually means
something's not quite right with notifications somewhere, making
"warn" probably at least as appropriate as "info" in more cases than
not.
  • Loading branch information
chrisbobbe committed Oct 5, 2023
1 parent 36f4466 commit c416abf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/settings/NotifTroubleshootingScreen.js
Original file line number Diff line number Diff line change
Expand Up @@ -408,7 +408,7 @@ export default function NotifTroubleshootingScreen(props: Props): React.Node {
break;
case MailComposer.MailComposerStatus.SENT:
showToast(_('Email sent'));
logging.info('NotifTroubleshootingScreen: MailComposer reports a sent email.');
logging.warn('NotifTroubleshootingScreen: MailComposer reports a sent email.');
break;
case MailComposer.MailComposerStatus.UNDETERMINED:
logging.warn('MailComposerStatus.UNDETERMINED');
Expand Down

0 comments on commit c416abf

Please sign in to comment.