-
Notifications
You must be signed in to change notification settings - Fork 124
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
Changed random timestamp used for props.key #58
Conversation
@jstejada Are you good with the change or would you like me to change something? |
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.
This looks good to me, although I'm not sure we should be updating node-sass in this commit
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.
Change Request: reverted back to older version of node-sass
@jstejada I reverted back to the older version of node-sass. Let me know if I need to make any more changes. |
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.
@cryptoads looks godo to me! only final change is to remove the change to package.json and yarn.lock.
Thanks!
@jstejada finally pushed changes to my branch with changes to package.json and yarn.lock removed. |
🎉 |
I made small changes to the cloneElement function in Utils.js. Instead of using Date.now() I used getUTCMilliSeconds() and it seems to have fixed the issue for me.
I was able to reproduce the "unique-key" issue easily with the original method by adding about 25
tags.
After changes I was unable to reproduce the issue.