Add logging for when we cancel verification (why, etc) #12554
Labels
T-Task
Tasks for the team like planning
Comments
|
@bwindels You've recently added some logging for verif, is this covered by that? |
|
(if it is, the logging isn't obvious) |
|
Hmm, possibly. It should probably be logged when receiving the .cancel event on each side of the request. We don't uniformly log the content when sending a cancellation though, which you could do by implementing it both in |
t3chguy
pushed a commit
that referenced
this issue
Oct 17, 2024
Co-authored-by: github-merge-queue <[email protected]>
JohnSimonsen
pushed a commit
to verji/element-web-v2
that referenced
this issue
Dec 20, 2024
* Upgrade dependency to [email protected] * v3.100.0-rc.0 * Upgrade dependency to [email protected] * Remove code smell assertion identified by Sonar (element-hq#12547) * This assertion is unnecessary since the receiver accepts the original type of the expression * Implement `client.getDomain()` null check * Update comment since `AutoDiscovery.findClientConfig` may still throw * Report verification and recovery state to posthog (element-hq#12516) * Report verification and recovery state to posthog * Fix CryptoApi import * Fix js-sdk import * Review: Use DeviceVerificationStatus instead of CrossSigningStatus * Review: Clean condition to check secrets in 4S * review: Fix redundent !! * Deprecate `Tooltip.tsx` * Use tooltip compound in `MessageComposer.tsx` * Use tooltip compound in `ReadReceiptGroup` component * Use tooltip compound in `ReadReceiptPerson` component * Delete `useTooltip.tsx` * Use compound tooltip in `ReactionRowButton` * Upgrade dependency to [email protected] * Migrate `TooltipTarget` usage to compound `Tooltip` (element-hq#12542) * Use new tooltip in `RoomTopic.tsx` * Use new tooltip in `MLocationBody.tsx` * Fix room topic * Update location snapshot * Use new tooltip in `AppPermission.tsx` * Remove `TooltipTarget` * Add tests for `RoomTopic` * Add tests for `ReadReceiptGroup.tsx` * Add more tests for `ReadReceiptGroup.tsx` * Add more test for `ReactionsRowButton.tsx` * Add seedrandom dep to `resolutions` (element-hq#12552) * Move seedrandom dep to `resolutions` ... to shut renovate up * Reinstate `seedrandom` as dev dependency If we just declare it in `resolutions`, it doesn't get installed at all, because it is only a devDependency of `bloom-filters`. * [create-pull-request] automated change (element-hq#12554) Co-authored-by: github-merge-queue <[email protected]> * Avoid using deprecated exports, fields, and duplicate code (element-hq#12555) * Fix tabbedview breakpoint width (element-hq#12556) Which should be 1024 according to the designs, not 768 * Tooltip: close field tooltip when ESC is pressed (element-hq#12553) * Close field tooltip when ESC is pressed * Use `Key.ESCAPE` * Close the release announcement when a dialog is opened (element-hq#12559) * Fire `ModalManagerEvent.Closed` when a dialog is closed * Listen to modal events in the RA * Fix first RA test * Upgrade dependency to [email protected] * v3.100.0-rc.1 * Upgrade dependency to [email protected] * v3.100.0 --------- Co-authored-by: RiotRobot <[email protected]> Co-authored-by: Ed Geraghty <[email protected]> Co-authored-by: Valere <[email protected]> Co-authored-by: Florian Duros <[email protected]> Co-authored-by: Florian Duros <[email protected]> Co-authored-by: David Baker <[email protected]> Co-authored-by: Richard van der Hoff <[email protected]> Co-authored-by: github-merge-queue <[email protected]> Co-authored-by: Michael Telatynski <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: