Skip to content

feat: remove ed from org#412

Merged
pataruco merged 1 commit intomainfrom
remove-ed
Apr 18, 2026
Merged

feat: remove ed from org#412
pataruco merged 1 commit intomainfrom
remove-ed

Conversation

@pataruco
Copy link
Copy Markdown
Contributor

No description provided.

@pataruco pataruco self-assigned this Apr 18, 2026
@pataruco pataruco requested a review from Copilot April 18, 2026 07:31
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Apr 18, 2026

View your CI Pipeline Execution ↗ for commit 5332492

Command Status Duration Result
nx affected run-many -t test --base=4ab706faf16... ✅ Succeeded <1s View ↗
nx affected run-many -t lint format --base=4ab7... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-04-18 07:32:33 UTC

Copy link
Copy Markdown

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

Removes the “Executive Director” content block from the Organization page data model and query, and updates footer contact details.

Changes:

  • Removed Executive Director heading/paragraph IDs from the models.
  • Removed Executive Director fields/variables/collection from the Organization page GraphQL query (and regenerated types).
  • Updated the footer mailing address and contact email.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
website/src/models/paragraphs/index.ts Removes the Executive Director paragraphs ID constant.
website/src/models/headings/index.ts Removes the Executive Director heading ID constant.
website/src/graphql/queries/pages/organization/index.graphql Drops Executive Director variables and query selections.
website/src/graphql/queries/pages/organization/index.generated.ts Updates generated types/document to match the updated Organization query.
website/src/components/footer/markup.tsx Updates displayed mailing address and mailto link.
website/src/app/[lang]/organization/page-body.tsx Removes Executive Director variables/section rendering, but still contains a remaining reference (see PR comment).
Comments suppressed due to low confidence (1)

website/src/app/[lang]/organization/page-body.tsx:67

  • executiveDirectorHeading is still being destructured from data and included in the headings array, but the Organization page GraphQL query no longer fetches an executiveDirectorHeading field. This will break TypeScript types (and makes the first heading entry always empty/undefined). Remove executiveDirectorHeading from the destructuring and from the headings array (and any other remaining references).
  const {
    executiveDirectorHeading,
    boardHeading,
    boardParagraphs,
    volunteersHeading,
    scientificHeading,
    researchHeading,
    scientificParagraphs,
    researchParagraphs,
    boardMembers,
    volunteersMembers,
    scientificMembers,
    researchMembers,
  } = data;

  const headings = [
    executiveDirectorHeading?.content ?? '',
    boardHeading?.content ?? '',
    volunteersHeading?.content ?? '',
    scientificHeading?.content ?? '',
    researchHeading?.content ?? '',

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

@pataruco pataruco merged commit 290b14d into main Apr 18, 2026
12 checks passed
@pataruco pataruco deleted the remove-ed branch April 18, 2026 07:33
pataruco added a commit that referenced this pull request Apr 18, 2026
🤖 I have created a release *beep* *boop*
---


## [3.9.0](v3.8.0...v3.9.0)
(2026-04-18)


### Features

* remove ed from org
([5332492](5332492))
* remove ed from org
([#412](#412))
([290b14d](290b14d))


### Bug Fixes

* org page build
([65b6719](65b6719))
* org page build ([#414](#414))
([e11a512](e11a512))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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