Skip to content

Team based requests.#487

Open
seanmarcia wants to merge 1 commit into
github:mainfrom
csmlo:pods
Open

Team based requests.#487
seanmarcia wants to merge 1 commit into
github:mainfrom
csmlo:pods

Conversation

@seanmarcia
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 7, 2026 17:54
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

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new “For Teams” experience that lists nonprofit team volunteer requests, including data generation during builds and new UI components/styles.

Changes:

  • Introduces an Organization model, /teams page, and organization listing UI with filters and expandable details
  • Adds a build-time generator (generate-organizations.ts) and wires it into prebuild
  • Introduces/updates styling and formatting (Tailwind/Next config formatting, new SCSS rules, new ESLint flat config)

Reviewed changes

Copilot reviewed 30 out of 34 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
types.ts Adds Organization interface for the new teams/orgs feature
pages/teams.tsx New Teams page rendering hero + organization list from generated data
components/Organization*.tsx New list/item/metadata components for organizations + filtering/expand behavior
generate-organizations.ts New build-time script to fetch and generate organizations.json
package.json Runs org generation as part of prebuild
styles/globals.scss Adds styles for org UI and changes global paragraph styling
topics.json Updates topics list content (now includes duplicates)
eslint.config.mjs Adds ESLint v9 flat config setup
components/Header/* Adds header-level tabs for Individuals vs Teams
tailwind.config.js, next.config.js, misc components Formatting/consistency updates
Comments suppressed due to low confidence (2)

styles/globals.scss:1

  • This introduces a global html p { ... } rule (because it’s nested under base, html) that removes margins and forces font sizing/line-height for all paragraphs across the site. That’s a broad styling change and is likely to cause regressions in existing layouts/components. Scope these paragraph rules to the new teams/org UI container(s) (e.g., .org-list-wrap p or the specific component classes) instead of globally.
    topics.json:1
  • topics.json now contains many duplicates (e.g., sdg-4, sdg-16, sdg-3, sdg-5). If this file is used to drive UI options or topic processing, duplicates will cause repeated options/entries and unstable behavior. Update the generator (or the file) to dedupe and produce a stable ordering (e.g., unique + sorted) before writing.

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

Comment thread components/OrganizationItem.tsx Outdated
Comment thread package.json
Comment thread eslint.config.mjs
Comment thread eslint.config.mjs
Comment thread components/RepositoryDescription.tsx Outdated
Comment thread components/OrganizationItemTopBar.tsx Outdated
Comment thread components/PageTabs.tsx Outdated
@seanmarcia seanmarcia closed this Jun 8, 2026
@seanmarcia seanmarcia reopened this Jun 8, 2026
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.

3 participants