Skip to content

Facebook links: display profile instead of full url#16414

Merged
charlesBochet merged 3 commits intomainfrom
facebook-links
Dec 12, 2025
Merged

Facebook links: display profile instead of full url#16414
charlesBochet merged 3 commits intomainfrom
facebook-links

Conversation

@Bonapara
Copy link
Copy Markdown
Member

@Bonapara Bonapara commented Dec 9, 2025

Like we do for LinkedIn/Twitter

Copilot AI review requested due to automatic review settings December 9, 2025 08:57
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@Bonapara Bonapara changed the title Add support for Facebook links to display the profile instead of full url Facebook links: display profile instead of full url Dec 9, 2025
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 7 files

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for Facebook links to display the profile/page identifier instead of the full URL, following the existing pattern used for LinkedIn and Twitter links.

Key changes:

  • Added Facebook type to the LinkType enum
  • Implemented Facebook URL detection and profile extraction logic
  • Updated all display components to handle Facebook links as social links

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/twenty-ui/src/navigation/link/components/SocialLink.tsx Added Facebook to the LinkType enum
packages/twenty-ui/src/utilities/utils/getDisplayValueByUrlType.ts Added Facebook URL parsing logic to extract profile identifier
packages/twenty-front/src/utils/checkUrlType.ts Added Facebook URL detection pattern
packages/twenty-front/src/utils/getDisplayValueByUrlType.ts Added Facebook URL parsing logic (duplicate of twenty-ui version)
packages/twenty-front/src/modules/ui/field/display/components/URLDisplay.tsx Updated condition to treat Facebook URLs as social links
packages/twenty-front/src/modules/ui/field/display/components/LinksDisplay.tsx Updated condition to render Facebook links with SocialLink component
packages/twenty-front/src/modules/ui/field/display/components/LinkDisplay.tsx Added Facebook detection and updated condition for social link rendering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:1455

This environment will automatically shut down when the PR is closed or after 5 hours.

@charlesBochet charlesBochet merged commit 7f50486 into main Dec 12, 2025
64 checks passed
@charlesBochet charlesBochet deleted the facebook-links branch December 12, 2025 17:50
@twenty-eng-sync
Copy link
Copy Markdown

Hey @charlesBochet! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

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.

4 participants