Skip to content

Conversation

lisamartin00
Copy link
Contributor

These changes resolve #58

@tocker
Copy link
Contributor

tocker commented Jul 26, 2018

Nice catch!
Looks like part of the problem is that the img tag is rendered before calculating the url - we'll need to re-visit this.

@tocker tocker merged commit 01625aa into cloudinary:master Jul 26, 2018
@lisamartin00
Copy link
Contributor Author

Hey @tocker thanks so much for merging this to master - are there plans to publish these changes to npm?

@tocker
Copy link
Contributor

tocker commented Aug 19, 2018

Yes, I'll publish it soon.

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.

Don't render on empty string for this.state.url

2 participants