Skip to content

Pull request for changes - #95

Merged
navveenb merged 6 commits into
nb-branchfrom
main
Jul 30, 2026
Merged

Pull request for changes#95
navveenb merged 6 commits into
nb-branchfrom
main

Conversation

@navveenb

Copy link
Copy Markdown
Contributor

Pull request for changes

claude and others added 6 commits July 30, 2026 08:56
Adds "Chair" and "Vice Chair" as recognized Role for Subscription
values in the Notion fetch pipeline, mapping to "Project Lead (Chair)"
and "Vice Chair" labels so standards pages (e.g. SWI) can distinguish
a project Chair from a Vice Chair instead of showing both as "Lead".
The Role for Subscription values in Notion had drifted to "Project
Chair" / "Project Vice Chair" (retyping a select value creates a new
option rather than reusing one), which the previous "Chair"/"Vice
Chair" mapping didn't match — causing the SWI Project Leadership
section to disappear entirely. Recognize both variants and document
the pitfall.
Notion's subscription query order isn't guaranteed, so the leads
array from projects.json could list the Vice Chair before the Chair
depending on query results. Sort by role rank in the page itself so
the Chair (Yi Ding) is always shown first/leftmost.
SWI appointment updates
…ojects

The standards index page and SEI standards page looked up the project by
slug "see", but Notion's PWCIs database has the SEI project's slug field
set to "sei". The mismatch meant projects.find() never matched, silently
dropping the SEI card from /standards/ and leaving its hero badges and
project leadership empty on /standards/sei/.
…standards-box-sja680

Fix SEI project slug mismatch so it appears in Current Standards & Projects
@netlify

netlify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Deploy Preview for green-software-foundation ready!

Name Link
🔨 Latest commit 7ff2084
🔍 Latest deploy log https://app.netlify.com/projects/green-software-foundation/deploys/6a6b65d40f039d00074c3845
😎 Deploy Preview https://deploy-preview-95--green-software-foundation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 87 (🔴 down 1 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@navveenb
navveenb merged commit ce792c8 into nb-branch Jul 30, 2026
6 of 7 checks passed
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