Closed sdotson closed 7 years ago
@dorono, good eye! Since the buttons don't work with localhost, I temporarily put in an arbitrary image url (the logo on staging/heroku server) to do the screenshot. The code as it stands now in this PR should be taking the featuredImageUrl
being used in the EventDetails component and passing it down as a prop to SocialBtns
Great job! Thanks.
I've addressed all comments but noticed something with the new facebook app id: The share dialogue won't work with everybody unless we make the app public. If it's too early for that we can designate a few testers for the app and it'll work for them.
Below are screenshots of what I see when I click facebook share now and where one could change the setting to make the app public in facebook (using a screenshot of my test app as an example).
Try it now.
Looks like that error is gone now! @pdw207
Had to whitelist the domain name. Now it works on staging.
For #97 .