Add separate page for each Rust team member and alumni

Author: Kobzol

locales/en-US/governance.ftl

@@ -34,7 +34,12 @@ governance-members-header = Members
 governance-alumni-header = Alumni
 governance-alumni-thanks = We also want to thank all past members for their invaluable contributions!
 
-## governance/all-team-members.mbs
+## governance/all-team-members.hbs
 governance-all-team-members-title = All Rust team members
 governance-all-team-members-intro = This section lists the members of currently active Rust teams.
 governance-all-team-members-alumni-intro = This section lists our team alumni.
+
+## govenance/person.hbs
+governance-person-title = Rust Project team member
+governance-person-team-member = Team member
+governance-person-team-alumni = Alumni

src/render.rs

@@ -180,11 +180,14 @@ pub fn render_index(render_ctx: &RenderCtx) -> anyhow::Result<()> {
 pub fn render_governance(render_ctx: &RenderCtx) -> anyhow::Result<()> {
     let data = render_ctx.teams.index_data();
 
+    // Index page
     for_all_langs("governance/index.html", |dst_path, lang| {
         render_ctx
             .page("governance/index", "governance-page-title", &data, lang)
             .render(dst_path)
     })?;
+
+    // Individual teams
     for team in data.teams {
         let data: PageData = render_ctx
             .teams
@@ -207,6 +210,7 @@ pub fn render_governance(render_ctx: &RenderCtx) -> anyhow::Result<()> {
         )?;
     }
 
+    // Archived teams
     let archived_teams_data = render_ctx.teams.archived_teams();
     for_all_langs("governance/archived-teams.html", |dst_path, lang| {
         render_ctx
@@ -219,6 +223,7 @@ pub fn render_governance(render_ctx: &RenderCtx) -> anyhow::Result<()> {
             .render(dst_path)
     })?;
 
+    // Page with all team members
     let all_team_members_data = render_ctx.teams.all_team_members();
     for_all_langs("governance/all-team-members.html", |dst_path, lang| {
         render_ctx
@@ -231,6 +236,28 @@ pub fn render_governance(render_ctx: &RenderCtx) -> anyhow::Result<()> {
             .render(dst_path)
     })?;
 
+    // A specific page for each team member
+    let all_person_data = render_ctx.teams.all_person_data();
+    for person in all_person_data {
+        // Use <username>/index.html for a nicer URL (/people/foo vs /people/foo.html).
+        for_all_langs(
+            &format!(
+                "governance/people/{}/index.html",
+                person.github.to_lowercase()
+            ),
+            |dst_path, lang| {
+                render_ctx
+                    .page(
+                        "governance/person",
+                        "governance-person-title",
+                        &person,
+                        lang,
+                    )
+                    .render(dst_path)
+            },
+        )?;
+    }
+
     Ok(())
 }
 

src/teams.rs

@@ -71,15 +71,11 @@ impl RustTeams {
             .filter(|team| team.website_data.is_some())
             // On the main page, show the leadership-council and all top-level
             // teams.
-            .filter(|team| team.kind == TeamKind::Team && team.subteam_of.is_none())
+            .filter(|team| is_toplevel_team(team))
             .map(|team| IndexTeam {
                 section: kind_to_str(team.kind),
                 page_name: team.website_data.clone().unwrap().page,
-                url: format!(
-                    "{}/{}",
-                    kind_to_str(team.kind),
-                    team.website_data.as_ref().unwrap().page
-                ),
+                url: get_team_relative_url(&team),
                 team,
             })
             .collect::<Vec<IndexTeam>>();
@@ -234,6 +230,113 @@ impl RustTeams {
 
         AllTeamMembers { active, alumni }
     }
+
+    pub fn all_person_data(&self) -> Vec<PersonData> {
+        let mut people: HashMap<String, PersonData> = HashMap::new();
+
+        enum TeamMode {
+            Member,
+            Alumni,
+            MemberOfArchivedTeam,
+        }
+
+        fn add_team(
+            people: &mut HashMap<String, PersonData>,
+            ctx: &RustTeams,
+            member: &TeamMember,
+            team: &Team,
+            mode: TeamMode,
+        ) {
+            let person = people
+                .entry(member.github.clone())
+                .or_insert_with(move || PersonData {
+                    name: member.name.clone(),
+                    github: member.github.clone(),
+                    active_teams: vec![],
+                    alumni_teams: vec![],
+                });
+            let teams = match mode {
+                TeamMode::Member => &mut person.active_teams,
+                TeamMode::Alumni | TeamMode::MemberOfArchivedTeam => &mut person.alumni_teams,
+            };
+            let url = match mode {
+                TeamMode::Member | TeamMode::Alumni => ctx.get_toplevel_team_url(team),
+                TeamMode::MemberOfArchivedTeam => Some("archived-teams.html".to_string()),
+            };
+            teams.push(PersonTeam::new(team, url));
+        }
+
+        for team in &self.archived_teams {
+            for member in team.members.iter().chain(&team.alumni) {
+                add_team(
+                    &mut people,
+                    self,
+                    member,
+                    team,
+                    TeamMode::MemberOfArchivedTeam,
+                );
+            }
+        }
+        for team in &self.teams {
+            if team.name == "all" || team.name == "alumni" || team.name == "leads" {
+                continue;
+            }
+
+            for member in &team.members {
+                add_team(&mut people, self, member, team, TeamMode::Member);
+            }
+            for member in &team.alumni {
+                add_team(&mut people, self, member, team, TeamMode::Alumni);
+            }
+        }
+
+        let mut people: Vec<PersonData> = people.into_values().collect();
+        people.sort_by(|a, b| a.github.cmp(&b.github));
+
+        for person in &mut people {
+            person
+                .active_teams
+                .sort_by(|a, b| a.webpage_name.cmp(&b.webpage_name));
+            person
+                .alumni_teams
+                .sort_by(|a, b| a.webpage_name.cmp(&b.webpage_name));
+        }
+
+        people
+    }
+
+    fn get_toplevel_team_url<'a>(&'a self, mut team: &'a Team) -> Option<String> {
+        while !is_toplevel_team(team) {
+            let Some(parent) = &team.subteam_of else {
+                return None;
+            };
+            let Some(parent) = self.teams.iter().find(|t| t.name == *parent) else {
+                return None;
+            };
+            team = parent;
+        }
+
+        if team.website_data.is_some() {
+            Some(get_team_relative_url(team))
+        } else {
+            None
+        }
+    }
+}
+
+/// Get a relative URL of a team that should be appended to
+/// Should only be used for top-level teams.
+fn get_team_relative_url(team: &Team) -> String {
+    assert!(is_toplevel_team(team));
+    format!(
+        "{}/{}",
+        kind_to_str(team.kind),
+        team.website_data.as_ref().unwrap().page
+    )
+}
+
+fn is_toplevel_team(team: &Team) -> bool {
+    team.kind == TeamKind::Team && team.subteam_of.is_none()
 }
 
 #[derive(Serialize)]
@@ -272,6 +375,49 @@ pub struct AllTeamMembers {
     alumni: Vec<TeamMember>,
 }
 
+#[derive(Serialize)]
+pub struct PersonTeam {
+    team: Team,
+    toplevel_url: Option<String>,
+    webpage_name: String,
+}
+
+impl PersonTeam {
+    fn new(team: &Team, toplevel_url: Option<String>) -> Self {
+        let webpage_name = team
+            .website_data
+            .as_ref()
+            .map(|w| w.name.clone())
+            .unwrap_or_else(|| {
+                // Turn inside-rust-reviewers into Inside Rust Reviewers
+                team.name
+                    .split("-")
+                    .map(|p| {
+                        p.chars()
+                            .take(1)
+                            .flat_map(|c| c.to_uppercase())
+                            .chain(p.chars().skip(1))
+                            .collect::<String>()
+                    })
+                    .collect::<Vec<String>>()
+                    .join(" ")
+            });
+        Self {
+            team: team.clone(),
+            toplevel_url: toplevel_url,
+            webpage_name,
+        }
+    }
+}
+
+#[derive(Serialize)]
+pub struct PersonData {
+    name: String,
+    pub github: String,
+    active_teams: Vec<PersonTeam>,
+    alumni_teams: Vec<PersonTeam>,
+}
+
 pub fn load_rust_teams() -> anyhow::Result<RustTeams> {
     println!("Downloading Team API data");
 

templates/governance/person.html.hbs

@@ -0,0 +1,62 @@
+{{#*inline "teams"}}
+  {{#each teams as |team|}}
+    <li>
+      {{#if team.toplevel_url }}
+        <a href="{{../baseurl}}/governance/{{team.toplevel_url}}{{ team-anchor team.team }}">
+          {{ team.webpage_name }}
+        </a>
+      {{else}}
+        {{ team.webpage_name }}
+      {{/if}}
+    </li>
+  {{/each}}
+{{/inline}}
+
+{{#*inline "page"}}
+  <section class="green" style="padding-bottom: 10px;">
+    <div class="w-100 mw-none mw-8-m mw9-l center f2">
+      <div class="w-100 w-50-l mb3 flex flex-row items-center">
+        <a class="mr4 w4 h4 flex-shrink-0" href="https://github.com/{{data.github}}">
+          <img class="w-100 h-100 bg-white br2"
+               src="https://avatars.githubusercontent.com/{{data.github}}"
+               alt="{{data.name}}">
+        </a>
+        <div>
+          {{data.name}}
+          <div class="f4">
+            GitHub: <a href="https://github.com/{{data.github}}">{{data.github}}</a>
+          </div>
+        </div>
+      </div>
+    </div>
+  </section>
+
+  {{# if data.active_teams }}
+    <section class="purple" style="padding-bottom: 10px;">
+      <div class="w-100 mw-none mw-8-m mw9-l center">
+        <header>
+          <h2>{{ fluent "governance-person-team-member" }}</h2>
+          <div class="highlight"></div>
+        </header>
+        <ul>
+          {{> teams teams=data.active_teams }}
+        </ul>
+      </div>
+    </section>
+  {{/if}}
+
+  {{# if data.alumni_teams }}
+    <section class="red" style="padding-bottom: 10px;">
+      <div class="w-100 mw-none mw-8-m mw9-l center">
+        <header>
+          <h2>{{ fluent "governance-person-team-alumni" }}</h2>
+          <div class="highlight"></div>
+        </header>
+        <ul>
+          {{> teams teams=data.alumni_teams }}
+        </ul>
+      </div>
+    </section>
+  {{/if}}
+{{/inline}}
+{{~> (lookup this "parent")~}}