Skip to content

Update team.json - add new gardeners #1728

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
+33 −0
Open

Update team.json - add new gardeners #1728

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions public/team.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1218,6 +1218,39 @@
"website": "https://pbhak.dev/",
"avatar": "https://cachet.dunkirk.sh/users/U07V1ND4H0Q/r"
},
{
"name": "Kat",
"department": "Welcoming",
"role": "Welcomer",
"acknowledged": false,
"bio": "Kat is a Hack Clubber who welcomes people into the Slack and helps them settle in.",
"slackId": "U06SQJ508LF",
"email": "",
"website": "",
"avatar": "https://cachet.dunkirk.sh/users/U06SQJ508LF/r"
},
{
"name": "Maadhav",
"department": "Welcoming",
"role": "Welcomer",
"acknowledged": false,
"bio": "Maadhav is a Hack Clubber from India who welcomes people into the Slack and helps them settle in.",
"slackId": "U08L7HLBT6V",
"email": "",
"website": "https://maadhavbhatt.github.io",
"avatar": "https://cachet.dunkirk.sh/users/U08L7HLBT6V/r"
},
{
"name": "Aishaani",
"department": "Welcoming",
"role": "Welcomer",
"acknowledged": false,
"bio": "Aishaani is a Hack Clubber who welcomes people into the Slack and helps them settle in.",
"slackId": "U097UCZE2BB",
"email": "",
"website": "",
"avatar": "https://cachet.dunkirk.sh/users/U097UCZE2BB/r"
Comment on lines +1243 to +1252
Copy link

@sentry sentry bot Nov 29, 2025

Choose a reason for hiding this comment

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

Bug: A team member with slackId U097UCZE2BB is listed twice in team.json under different departments.
Severity: CRITICAL | Confidence: High

🔍 Detailed Analysis

The team.json file contains two entries for Aishaani with the identical slackId U097UCZE2BB. One entry is under the "Events" department, and the other is under the "Welcoming" department. The pages/team.tsx component filters team members by department without any deduplication logic based on slackId. This will result in Aishaani being displayed twice on the team page, once in each department section, leading to redundant and confusing information for users and violating repository guidance.

💡 Suggested Fix

Consolidate the two entries for Aishaani (slackId U097UCZE2BB) into a single entry, updating her department and role as intended, or remove one of the duplicate entries.

🤖 Prompt for AI Agent 
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: public/team.json#L1243-L1252

Potential issue: The `team.json` file contains two entries for `Aishaani` with the
identical `slackId` `U097UCZE2BB`. One entry is under the "Events" department, and the
other is under the "Welcoming" department. The `pages/team.tsx` component filters team
members by department without any deduplication logic based on `slackId`. This will
result in `Aishaani` being displayed twice on the team page, once in each department
section, leading to redundant and confusing information for users and violating
repository guidance.

Did we get this right? 👍 / 👎 to inform future reviews.
Reference ID: 4332633

},
{
"name": "Joel Gallagher",
"department": "Welcoming",
Expand Down