Skip to content

feat(social): add telegram profile link support #968

Open
AntonyLeons wants to merge 2 commits intorahuldkjain:mainfrom
AntonyLeons:main
Open

feat(social): add telegram profile link support #968
AntonyLeons wants to merge 2 commits intorahuldkjain:mainfrom
AntonyLeons:main

Conversation

@AntonyLeons
Copy link
Copy Markdown

@AntonyLeons AntonyLeons commented Apr 26, 2026

🔄 Pull Request

📋 Type of Change (check all applicable)

  • 🐛 Bug Fix - Fixes an issue without breaking existing functionality
  • [ x] ✨ Feature - Adds new functionality
  • Performance - Improves performance without changing functionality
  • ♻️ Refactor - Code changes that neither fix bugs nor add features
  • 📚 Documentation - Updates to documentation, comments, or README
  • 🎨 Style - Code style changes (formatting, missing semi-colons, etc.)
  • 🧪 Test - Adding or updating tests
  • 🏗️ Build - Changes to build system or dependencies
  • 👷 CI/CD - Changes to CI/CD workflows
  • 🔒 Security - Security improvements or vulnerability fixes
  • Accessibility - Improves accessibility compliance
  • 📱 Mobile - Mobile-specific improvements
  • 🌐 i18n - Internationalization changes

📖 Description

feat(social): add telegram profile link support

What changed?

Why was this change made?

How does this change help users?

🔗 Related Issues

  • Closes #
  • Fixes #
  • Related to #

🧪 Testing & Quality Assurance

Testing Done (check all applicable)

  • Manual testing - Tested functionality manually
  • 🧪 Unit tests - Added/updated unit tests
  • 🔄 Integration tests - Tested with other components
  • 📱 Mobile testing - Tested on mobile devices
  • Accessibility testing - Tested with screen readers/keyboard nav
  • 🌐 Cross-browser testing - Tested in multiple browsers
  • 🎨 Visual testing - Checked UI/UX in light/dark themes

Test Instructions

Expected Behavior

📸 Screenshots/Recordings

Before

After

📋 Checklist

Code Quality

  • 🔍 TypeScript - No TypeScript errors (npm run type-check)
  • 🧹 Linting - No ESLint errors (npm run lint)
  • 🎨 Formatting - Code is properly formatted (npm run format)
  • 🏗️ Build - Production build succeeds (npm run build)
  • Performance - No performance regressions introduced

Accessibility

  • WCAG Compliance - Follows WCAG 2.1 AA guidelines
  • ⌨️ Keyboard Navigation - All interactive elements are keyboard accessible
  • 🔍 Screen Reader - Proper ARIA labels and semantic HTML
  • 🎨 Color Contrast - Meets contrast requirements
  • 🎯 Focus Management - Visible focus indicators

Mobile & Responsive

  • 📱 Mobile Responsive - Works on mobile devices (320px+)
  • 🖥️ Desktop - Works on desktop (1024px+)
  • 📐 Tablet - Works on tablet sizes (768px+)
  • 🔄 Orientation - Works in portrait and landscape

Browser Compatibility

  • 🌐 Chrome - Latest version
  • 🦊 Firefox - Latest version
  • 🧭 Safari - Latest version
  • 📱 Mobile Safari - iOS Safari
  • 📱 Chrome Mobile - Android Chrome

Documentation

  • 📚 Code Comments - Added helpful comments for complex logic
  • 📖 Documentation - Updated relevant documentation
  • 📝 README - Updated README if needed
  • 🔄 Changelog - Will be auto-generated from conventional commits

Security & Privacy

  • 🔒 No Secrets - No API keys, passwords, or sensitive data exposed
  • 🛡️ Input Validation - Proper validation for user inputs
  • 🔐 XSS Prevention - Protected against XSS attacks
  • 🍪 Privacy Compliant - Follows GDPR/privacy requirements

🚀 Deployment Notes

  • No breaking changes - Backward compatible
  • Database changes - N/A (static site)
  • Environment variables - No new env vars needed
  • Third-party dependencies - No new external dependencies

📝 Additional Notes

👀 Reviewers


By submitting this PR, I confirm that:

Add Telegram as a new social media platform option for user profiles. This includes:
- Adding Telegram field to the SocialLinks interface and default constants
- Including validation schema for the Telegram field
- Adding Telegram input field to the social section UI with appropriate icon and placeholder
- Providing a Telegram SVG icon in the social icons directory
- Updating markdown generator to handle Telegram links and use CDN icon for better compatibility
Remove the separate socialIconBaseUrl variable and inline the base URL
directly in the social icon markup generation. Also change the Telegram
icon from a direct CDN URL to use the local SVG filename for consistency
with other social icons.
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.

1 participant