apps icon indicating copy to clipboard operation
apps copied to clipboard

Update the display logic of the slashing page

Open kianenigma opened this issue 2 years ago • 1 comments

... IFF needed, since one storage items is being removed an the defintion of the keys in UnappliedSlashes is slightly changing, as per https://github.com/paritytech/substrate/pull/11823/files

kianenigma avatar Jul 27 '22 16:07 kianenigma

Removed here - https://github.com/polkadot-js/api/pull/5118/files#diff-fa159c3aa82e1d4780a5038af75643716aaaf0531ca608b31ce47199833a8159L1326

Used here - https://github.com/polkadot-js/apps/blob/master/packages/react-hooks/src/useAvailableSlashes.ts#L23

Dual support required.

jacogr avatar Jul 29 '22 14:07 jacogr

Logic is very different, so indeed 2 code-paths are to be used. Previous keyed on era-where-slash-occurred, now keyed on era-where-slash-is-applied which would mean the logic to determine the range is very different between the two (it would also need display adjustments since there is shows when the slash occurred)

jacogr avatar Nov 02 '22 06:11 jacogr

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.

polkadot-js-bot avatar Nov 22 '22 11:11 polkadot-js-bot