-
Notifications
You must be signed in to change notification settings - Fork 340
feat: detect links in user bio #1056
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: detect links in user bio #1056
Conversation
- Add parseUrls utility function to detect URLs in bio text - Convert detected URLs to clickable links with proper styling - Match email link styling for consistency - Support both http/https URLs and URLs without protocol
| import { css } from '@emotion/react'; | ||
| import { Box } from '@embeddedchat/ui-elements'; | ||
|
|
||
| const URL_REGEX = /(https?:\/\/[^\s<>"']+)/gi; |
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.
In EmbeddedChat, we might be using a parser library to handle link scenarios. It would be better to explore that option instead of relying on a custom parser, which could fail in various situations. Thanks.
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.
ok, Will look into this
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.
In EmbeddedChat, we might be using a parser library to handle link scenarios. It would be better to explore that option instead of relying on a custom parser, which could fail in various situations. Thanks.
Screencast.From.2026-01-15.15-27-34.mp4
Got it, updated
Spiral-Memory
left a comment
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
Make URLs in user bio clickable
Acceptance Criteria fulfillment
http://orhttps://in bio are automatically detectedtarget="_blank"andrel="noopener noreferrer")Implementation Details
styleprop matching email link pattern for consistencyparseUrls.jsfor URL detectionUserInformation.jsto use parseUrls for bio fieldFixes #1055
Screenshots
Light theme

Dark theme
PR Test Details
Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace
<pr_number>with the actual PR number.