Skip to content
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

Support non-nodejs environments #41

Merged

Conversation

lidawang
Copy link
Contributor

Hello! We've been using react-lazyload at our company for making React components lazyload both in a nodeJS and not in a nodeJS environment, and the latter one doesn't work because of the assumption that process is a defined variable. This PR just puts in a check to make sure the variable exists before accessing it and is pretty harmless.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.041% when pulling d6040db on lidawang:lw-support-non-node-environments into b299170 on jasonslyvia:master.

@jasonslyvia
Copy link
Collaborator

Cool, not thought of that, thanks for contribution!

@jasonslyvia jasonslyvia merged commit e08e687 into twobin:master Aug 17, 2016
@jasonslyvia
Copy link
Collaborator

Are you expecting an immediate release?

@lidawang
Copy link
Contributor Author

Hello! Thanks for merging this so fast! As for an immediate release, that would be welcome - although I might have another contribution coming in with checkVisible, once I debug and figure out what our issue is (something to do with how modal dialogs...), so feel free to wait for that.

@jasonslyvia
Copy link
Collaborator

I'd love to.

@lidawang
Copy link
Contributor Author

lidawang commented Sep 1, 2016

Er actually, the other bug I was looking into turned out to be a problem with our usage of this library - not a bug in it. In other words, a new release of this would be mighty appreciated so we can start using this fix!

@lidawang lidawang mentioned this pull request Sep 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants