Update Footer Social Media Links to RyloHQ#256
Conversation
Updated the LinkedIn URL from https://www.linkedin.com/company/sign-mt to https://www.linkedin.com/company/rylohq/ in the landing footer component and the docs VitePress config. Co-authored-by: Tomer Aharoni <tomeraharonii@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughTwo LinkedIn social link URLs are updated to https://www.linkedin.com/company/rylohq/: one in the VitePress documentation configuration theme social links, and another in the landing footer component template. ChangesLinkedIn Company URL Updates
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Updated the X/Twitter URL from https://x.com/NagishSign to https://x.com/rylohq in the landing footer component. Co-authored-by: Tomer Aharoni <tomeraharonii@users.noreply.github.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/app/pages/landing/landing-footer/landing-footer.component.html`:
- Line 28: The external anchor in landing-footer.component.html (the <a
href="https://www.linkedin.com/company/rylohq/" target="_blank">LinkedIn</a>
element) must include rel="noopener noreferrer" to mitigate reverse-tabnabbing;
update that anchor and any other sibling external links in the same list that
use target="_blank" to add rel="noopener noreferrer" so the behavior is
consistent across the component.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 4b4f8086-f44a-4eeb-baad-89791a97ce34
📒 Files selected for processing (2)
docs/.vitepress/config.mtssrc/app/pages/landing/landing-footer/landing-footer.component.html
| <ul> | ||
| <li><a href="https://x.com/NagishSign" target="_blank">Twitter (X)</a></li> | ||
| <li><a href="https://www.linkedin.com/company/sign-mt" target="_blank">LinkedIn</a></li> | ||
| <li><a href="https://www.linkedin.com/company/rylohq/" target="_blank">LinkedIn</a></li> |
There was a problem hiding this comment.
Add rel="noopener noreferrer" for external links opened in new tabs.
target="_blank" without rel="noopener noreferrer" exposes reverse-tabnabbing risk. Please add rel on this link (and keep it consistent for sibling external links in this list).
Suggested patch
- <li><a href="https://www.linkedin.com/company/rylohq/" target="_blank">LinkedIn</a></li>
+ <li><a href="https://www.linkedin.com/company/rylohq/" target="_blank" rel="noopener noreferrer">LinkedIn</a></li>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <li><a href="https://www.linkedin.com/company/rylohq/" target="_blank">LinkedIn</a></li> | |
| <li><a href="https://www.linkedin.com/company/rylohq/" target="_blank" rel="noopener noreferrer">LinkedIn</a></li> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/app/pages/landing/landing-footer/landing-footer.component.html` at line
28, The external anchor in landing-footer.component.html (the <a
href="https://www.linkedin.com/company/rylohq/" target="_blank">LinkedIn</a>
element) must include rel="noopener noreferrer" to mitigate reverse-tabnabbing;
update that anchor and any other sibling external links in the same list that
use target="_blank" to add rel="noopener noreferrer" so the behavior is
consistent across the component.
cursor
Bot
changed the title
|
Visit the preview URL for this PR (updated for commit b3a27d1): https://translate-sign-mt--pr256-cursor-update-linked-2a6ix5z3.web.app (expires Thu, 18 Jun 2026 14:52:15 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 739446cfe7a349700ebd347d2a39e3b90ba24425 |
AmitMY
left a comment
AmitMY
left a comment
There was a problem hiding this comment.
wrong repo - this is sign.mt, not https://rylo.com/sign/translate
@HTMHell let's update the linkedin as preferred here
Tomer - we will merge rylo-translate (private) here once this site is fully deprecated.
4 participants
Summary
Updates the footer social media links to point to RyloHQ profiles.
Changes
src/app/pages/landing/landing-footer/landing-footer.component.html— Updated the LinkedIn URL fromsign-mttorylohq, and the Twitter (X) URL fromNagishSigntorylohqdocs/.vitepress/config.mts— Updated the LinkedIn URL in the docs site social links torylohqResolves WWW-244
Slack Thread
Summary by CodeRabbit