SSR compatibility patch

[eghan]

Working with Gatsby the window property assignment works fine in development build but fails in deployment builds because 'window' is not available. Simple conditional assignment I am presuming preferable to componentDidMount style solutions as unintended side effects are minimized.

Thanks for this fantastic repo! works great(almost!) with Gatsby where all other paypal solutions seem to fail

[dderooy]

I have the same issue! Thanks for creating a PR for this. @thinhvo0108 can you approve this?

[thinhvo0108]

Thanks for your contribution and ideas, I will check and approve, merge as well as publish new version on NPM when I have time soon! Best,

…

On Wed, Jan 16, 2019 at 4:39 AM dderooy ***@***.***> wrote: I have the same issue! Thanks for creating a PR for this. @thinhvo0108 <https://github.com/thinhvo0108> can you approve this? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#37 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AE-TmJAXdNsO4hQw8dhp0kWrmWXy0CKrks5vDi6JgaJpZM4Y6WyX> .

[aralroca]

My question is: Why is necessary to add React and ReactDOM to window object? 😕At least, if this is absolute necessary... should be clean it on unmount...

componentWillUnMount(){
  delete window.React
  delete window.ReactDOM
}

[Flatheadman]

Yeah, componentDidMount style solution does work.