Skip to content

Conversation

@dynamicdhxx
Copy link

@dynamicdhxx dynamicdhxx commented Nov 25, 2025

Related Tickets & Documents

Fixes: #2965

Description

  • Fetch the reviewer’s canonical WordPress slug (nicename) via WPGraphQL and thread it through the UI so we stop guessing URLs from the display name.
  • Guard reviewer rendering when metadata isn’t available to avoid runtime errors, and pass the slug to both the hero hover card and the reviewer details card so “View all posts” opens the correct archive.

Changes

  • Add slug to getReviewAuthorDetails in lib/api.ts.
  • Store slug in technology/community post pages, propagate it to <PostHeaderAuthors> and <PostBody>, and treat reviewer data as optional.
  • Update PostHeaderAuthors and ReviewingAuthor components to build reviewer links from the canonical slug (fallback to sanitized name) so the reviewer card and button behave like the author card.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)

Testing

  • npm run dev
  • Manually verified reviewer cards on a technology and a community post: hover card and “View all posts” now route to the expected reviewer archive and load posts.

Demo

  • N/A (behavioral fix; no visual changes)

Environment and Dependencies

  • New Dependencies: None
  • Configuration Changes: None

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added corresponding tests
  • I have run the build/lint command to ensure there are no errors (npm run lint)
  • My changes have been tested across relevant browsers/devices (manually verified reviewer links)
  • For UI changes, I've included visual evidence (not applicable)

Signed-off-by: Dhruv Bharuka <[email protected]>

signed by Dhruv Bharuka
@amaan-bhati
Copy link
Member

Hey @kaushiktak19 Can you please do the first level of review and tag me here once the first level of review is done?

@dynamicdhxx
Copy link
Author

@amaan-bhati @kaushiktak19 the lighthouse check is failing due to a github token issue.

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.

2 participants