Skip to content

Add country to profiles on index page#456

Merged
jbampton merged 2 commits intoNextCommunity:mainfrom
jbampton:add-country-index
May 2, 2026
Merged

Add country to profiles on index page#456
jbampton merged 2 commits intoNextCommunity:mainfrom
jbampton:add-country-index

Conversation

@jbampton
Copy link
Copy Markdown
Member

@jbampton jbampton commented May 2, 2026

No description provided.

@jbampton jbampton requested a review from BaseMax as a code owner May 2, 2026 11:03
@jbampton jbampton self-assigned this May 2, 2026
@jbampton jbampton added this to the Turbo milestone May 2, 2026
@jbampton jbampton added this to Next May 2, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Next May 2, 2026
@jbampton jbampton moved this from In progress to Review in progress in Next May 2, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the homepage directory cards to show each developer’s country alongside their existing location, bringing the index page into alignment with the profile detail view and the repository’s profile data model.

Changes:

  • Read person.data.country in src/index.njk.
  • Render the location badge when either location or country exists.
  • Concatenate location and country with a comma when both are present.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jbampton jbampton merged commit 3a3e294 into NextCommunity:main May 2, 2026
13 of 14 checks passed
@jbampton jbampton deleted the add-country-index branch May 2, 2026 11:06
@github-project-automation github-project-automation Bot moved this from Review in progress to Done in Next May 2, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the person display in src/index.njk to include country information alongside the location. It introduces a new country variable and modifies the logic to display either the location, the country, or both separated by a comma within the badge. I have no feedback to provide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants