Skip to content

Refactor social link rendering into shared reusable component#573

Open
Rohan-Ver-ma wants to merge 2 commits intojenkins-infra:mainfrom
Rohan-Ver-ma:refactor/shared-social-links-component
Open

Refactor social link rendering into shared reusable component#573
Rohan-Ver-ma wants to merge 2 commits intojenkins-infra:mainfrom
Rohan-Ver-ma:refactor/shared-social-links-component

Conversation

@Rohan-Ver-ma
Copy link
Copy Markdown
Contributor

Description

This PR removes duplicated social link rendering logic by introducing a shared SocialLinks component and replacing repeated JSX blocks in the affected views.

Fixes #572

What this PR does

  • Adds a shared SocialLinks component as a single source of links.
  • Updates contributor card, search results, and contributor details to use the shared component.
  • Preserves existing behavior and visual output

Why this improves the codebase

  • Better maintainability: future social-link changes are made once.
  • Better consistency: same logic and styling path across pages.
  • Better readability: parent components are shorter and easier to understand.
  • Lower regression risk from duplicated UI logic.

Checklist

  • PR title is clear and descriptive
  • Changes follow project conventions
  • No unrelated changes included
  • Confirmed behavior remains functionally the same after refactor

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.

Refactor duplicated social link rendering into a shared component

1 participant