Page MenuHomePhabricator

Extend showNotification method to cover the case of decryption error notitication
ClosedPublic

Authored by marcin on Nov 23 2023, 8:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 30 2024, 9:21 PM
Unknown Object (File)
Oct 26 2024, 6:04 AM
Unknown Object (File)
Oct 21 2024, 4:49 AM
Unknown Object (File)
Oct 2 2024, 3:22 AM
Unknown Object (File)
Oct 2 2024, 3:22 AM
Unknown Object (File)
Oct 2 2024, 3:22 AM
Unknown Object (File)
Oct 2 2024, 3:22 AM
Unknown Object (File)
Oct 2 2024, 3:22 AM
Subscribers

Details

Summary

This differential refactors showNotification method in native desktop code so that it can handle future case of notification that we failed to decrypt. If
we fail to decrypt notification we will either display generic "Comm notification" or add error stack trace to the body depending on whether current user is a staff
member or we are running DEV environment.

Test Plan
  1. Modify listenForNotifications method so that on each notification it calls showNotification twice. The first time with real notification and teh

second time with notification that matched decryption error notification structure (error and displayErrorMessage fields)

  1. Check navigation behaviour of error message notification.

Diff Detail

Repository
rCOMM Comm
Branch
marcin/eng-5383
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Nov 24 2023, 6:03 AM