Skip to content

fix: resolve Aabid Sofi profile issue and make member status checks case-insensitive#7865

Merged
Rajesh-Nagarajan-11 merged 3 commits into
layer5io:masterfrom
rishiraj38:fix/aabid-profile-status
Jul 13, 2026
Merged

fix: resolve Aabid Sofi profile issue and make member status checks case-insensitive#7865
Rajesh-Nagarajan-11 merged 3 commits into
layer5io:masterfrom
rishiraj38:fix/aabid-profile-status

Conversation

@rishiraj38

Copy link
Copy Markdown
Member

Description

This PR fixes an issue where maintainer Aabid Sofi's profile did not render in the active members directory grid, and addresses rendering style issues on their profile card.

Root Causes:

  1. Aabid Sofi's profile status in aabid-sofi/index.mdx was set to active (lowercase a), whereas the site logic performed case-sensitive checks for Active (capitalized A).
  2. The case sensitivity also broke the profile card background color and inactive/active border styles.

Proposed Changes:

  • aabid-sofi/index.mdx: Capitalized status: active to status: Active.
  • MemberList.js: Updated status matching logic to be case-insensitive (status.toLowerCase() === badge.toLowerCase()) so that lowercase values do not prevent members from appearing in the grid.
  • ProfileCard.style.js: Updated styled-components condition checks to match status case-insensitively, ensuring consistent profile card style rendering.
  • Member-single/index.js: Standardized status check to be case-insensitive for rendering the profile-image-inactive class.
  • Meshmates/index.js: Standardized status check to filter active/inactive MeshMates case-insensitively.

Signed commits

  • Yes, I signed my commits.

…ensitive

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-07-13 07:32 UTC

@banana-three-join banana-three-join left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the change to Active is good enough for this PR and instead, a beginner issue can be created to integrate a Helper function which returns the output of the status but with the correct capitalization.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit e312232 into layer5io:master Jul 13, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants