Rename dangerouslySetInnerHTML to dangerousInnerHTML#2257
Rename dangerouslySetInnerHTML to dangerousInnerHTML#2257syranide wants to merge 1 commit intofacebook:masterfrom syranide:redih
Conversation
|
Note to self, rename #2258 after this is merged (there will be lots of conflicts). |
|
This doesn't deprecate nicely. We should make |
|
@zpao Will add. |
|
Yea I guess this is helpful because you get the error message an it is easy. If this wasn't easy we could just require a codemod since it's such a unique name.
|
|
@zpao I've re-added Why don't we have a utility helper for emitting warnings once per component and (optionally) with the component name automatically prefixed? ... Should I put up a PR for something like it? |
|
Ugh. We should've done this earlier. If we do it now, it's not worth the headache. Maybe if we intro |
From #1515 (comment)
Steps for upgrading existing codebases:
dangerouslySetInnerHTMLwithdangerousInnerHTML.cc @zpao @sebmarkbage