From 64c00a088908bff022005ac586c6b7201f4b5eeb Mon Sep 17 00:00:00 2001 From: Greg Littlefield Date: Mon, 23 Sep 2024 10:15:58 -0700 Subject: [PATCH] Address CR feedback --- doc/null_safety/null_safe_migration.md | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/doc/null_safety/null_safe_migration.md b/doc/null_safety/null_safe_migration.md index 7429bd1e6..a1c756fbd 100644 --- a/doc/null_safety/null_safe_migration.md +++ b/doc/null_safety/null_safe_migration.md @@ -206,10 +206,11 @@ of the null safety and required props docs for instructions on how to handle the #### connect -We'll be adding a codemod to help migrate the `connect` case: https://github.com/Workiva/over_react_codemod/issues/295 - -Alternatively, you could refactor your component to instead utilize [OverReact Redux hooks](./over_react_redux_documentation.md#hooks), -which avoid this problem by accessing store data and dispatchers directly in the component as opposed to passing it in via props. +For connect, either +- Disable validation using the instructions linked above + - Note: for now, this must be done manually, but we'll be adding a codemod to help do this automatically for `connect`: https://github.com/Workiva/over_react_codemod/issues/295 +- Refactor your component to instead utilize [OverReact Redux hooks](../over_react_redux_documentation.md#hooks), + which avoid this problem by accessing store data and dispatchers directly in the component as opposed to passing it in via props. #### Implementing abstract `Ref`s