-
Notifications
You must be signed in to change notification settings - Fork 392
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
try adding nodeRef #1140
base: master
Are you sure you want to change the base?
try adding nodeRef #1140
Conversation
if this works seems good to me, any reason it's still in a draft? |
@jquense looks like I need to refactor because its a class-based component. |
Is You can use |
Try running the docs but in strict mode? |
@quinn Did you have any luck? |
Any movement on this? Would love to clean that last error up on my logs :) |
@astralmedia PR should work, I haven't had a chance to run the docs in strict mode as advised |
also, I can't see why the build failed? says, expired |
getting this error when I try to run docs:
|
Hi, can this PR be merged or does it needs more work? |
attempt to fix #1137