feat: recovery queue #2832
feat: recovery queue #2832
Conversation
Branch preview⏳ Deploying a preview site... |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success1193 tests passing in 168 suites. Report generated by 🧪jest coverage report action from 8da39d0 |
src/hooks/useTimestamp.ts
Outdated
| @@ -0,0 +1,17 @@ | |||
| import { useState, useEffect } from 'react' | |||
|
|
|||
| export function useTimestamp(interval = 1_000): number { | |||
There was a problem hiding this comment.
I would call this hook useClock or useIntervalTimestamp. useTimestamp to me reads as a one-time timestamp.
There was a problem hiding this comment.
I've renamed it to useClock in 6896ea6.
|
|
||
| export const RecoveryInfo = (): ReactElement => { | ||
| return ( | ||
| <Tooltip title="This recovery attempt was not created via the interface" placement="top" arrow> |
There was a problem hiding this comment.
This recovery attempt was not created via the interface
That it wasn't created via the interface is a poor way to say it's a fraudulent transaction.
Design-wise, I wouldn't even display them as recovery attempts. Too easy to confuse with a legit recovery. The font should be red at least.
There was a problem hiding this comment.
I've renamed the transaction type to "Malicious transaction" (coloured in error.main) and changed the tooltip to "Suspicious activity" in 6896ea6.
What it solves
Resolves #2763
How this PR fixes it
This implements a new "Pending recovery" section to the transaction queue which displays all recovery attempts from connected recovery-specific Delay Modifiers, as well as associated adjustments:
executorandisMaliciousflags have been added, as well as all timestamps converted to ms)Note: text was tweaked and does not necessarily match the design. It is also not possible to skip recovery proposals yet. As such, queued recovery attempts and their "skip" transactions are not included in this PR.
How to test it
Screenshots
Old naming/colouring:
Checklist