-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing usages of $applyNodeReplacement
#3592
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
cb82b06
to
30ef5a8
Compare
30ef5a8
to
eaa5f54
Compare
$applyNodeReplacement
a2d1a1f
to
9d7c62d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally you don't need to add $applyNodeReplacement to your subclasses unless you plan to expose them as a library, which I think is probably rare in userland. I would say let's not put this in every creator on the playground for people to just copy and paste everywhere. The problem this is solving is allowing customization of core node behavior without changing the core library itself. The concept is not necessarily useful or applicable in places like TweetNode, where we don't even expose the node.
9d7c62d
to
6ad12e4
Compare
@acywatson Fair point. I've just added it in to the exported nodes now. |
No description provided.