-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: implement ShareDropdownMenu #39
Conversation
frontend/apps/service-site/src/features/posts/components/ShareDropdownMenu/CopyLinkItem.tsx
Show resolved
Hide resolved
> | ||
<span>{t('posts.share.facebook')}</span> | ||
</DropdownMenuItem> | ||
<DropdownMenuItem | ||
leftIcon={<LinkedInIcon className={styles.icon} />} | ||
onSelect={() => alert('Item 3 clicked')} | ||
onSelect={handleSelect( | ||
`https://www.linkedin.com/sharing/share-offsite/?url=${url}`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that if the page returns 200 statuses, as in this URL, it can be shared without any problem.
https://www.linkedin.com/sharing/share-offsite/?url=https://route06.co.jp
...tend/apps/service-site/src/features/posts/components/ShareDropdownMenu/ShareDropdownMenu.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: hoshino tsuyoshi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx!!
Share URLs for each SNS were implemented.
2024-10-11.18.20.23.mov
NOTE: linkedin seems to be an error for 401 pages.