matrix-react-sdk icon indicating copy to clipboard operation
matrix-react-sdk copied to clipboard

Typescript strict null checks pt1

Open Palid opened this issue 4 years ago • 2 comments

Removed some unused variables (mostly private ones) and changed the types accordingly to strictNullCheck in tsconfig.

Please don't mind failing eslint so far, we need to figure this out slightly better. Maybe change APIs in case of the SettingsStore too (make new API and deprecate current one?)

Related to https://github.com/vector-im/element-web/issues/21967


This change is marked as an internal change (Task), so will not be included in the changelog.

Palid avatar Aug 08 '21 18:08 Palid

It seems that @SimonBrandner wants to take over this one, so please do! 😄

Palid avatar Sep 02 '21 16:09 Palid

I don't have the time to handle this atm

SimonBrandner avatar Jan 07 '22 17:01 SimonBrandner

Thanks for this but it has rotted beyond usability, some of these sites have already been fixed, rest will be addressed as part of https://github.com/vector-im/element-web/issues/21967

t3chguy avatar Feb 03 '23 12:02 t3chguy