Opened this issue · 2 comments
I really love this plugin. Thanks for providing it. Pinterest requires two parts to their url. They have a media link and they have a url link. It appears we can only pass the link and it gets put in both. Is there any way to have it so that it will not pull both and instead have two parameters one for media and one for the url to redirect to? Or am I missing something?
The proposed changes I would like to see are in the consts.js
pinterest: (link = '', message = '', media = undefined) =>
https://pinterest.com/pin/create/button/?url=${encodeURIComponent( link )}&media=${encodeURIComponent(media ? media : link)}&description=${encodeURIComponent( message )},
Then in the buttons/pinterest.js
export default ({ link, message, media, ...props }) => (
<ButtonFactory
{...props}
name="Pinterest"
href={links.pinterest(link, message, media)}
buttonComponent={Pinterest}
iconFill={PinterestIconFill}
iconCircle={PinterestIconCircle}
iconCircleSolid={PinterestIconCircleSolid}
/>
)
I added the ternary statement so that it is backward compatible with all the other people using the plugin. So if they are using link for media then it will still be link because they have not set up media.
What are your thoughts?
Is this possible to update soon? thanks