Skip to content

Commit 33f4621

Browse files
authored
Merge pull request #1938 from ehuss/deploy
Deploy
2 parents b0327cd + b0d3b17 commit 33f4621

File tree

4 files changed

+33
-6
lines changed

4 files changed

+33
-6
lines changed

locales/en-US/governance.ftl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ governance-team-email = Email the { $teamname }
1919
governance-team-repository = { $teamname } repository
2020
governance-team-discord = { $channel } on Discord
2121
governance-team-zulip = { $stream } on Zulip
22+
governance-team-matrix-room = { $room } on Matrix
2223
governance-user-github = GitHub: { $link }
2324
governance-user-team-leader = Team leader
2425
governance-members-header = Members

src/redirect.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,11 @@ static PAGE_REDIRECTS: &[(&str, &str)] = &[
2424
("security.html", "policies/security"),
2525
("team.html", "governance"),
2626
("user-groups.html", "community"),
27+
// Team changes
28+
(
29+
"governance/teams/release",
30+
"governance/teams/infra#team-release",
31+
),
2732
];
2833

2934
static STATIC_FILES_REDIRECTS: &[(&str, &str)] = &[

src/teams.rs

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,16 @@ impl Data {
5959
self.teams
6060
.into_iter()
6161
.filter(|team| team.website_data.is_some())
62-
.filter(|team| matches!(team.subteam_of.as_deref(), None | Some("launching-pad")))
62+
// On the main page, show the leadership-council, all top-level
63+
// teams, and everything in the launching pad. We may want to
64+
// consider putting launching pad teams in a separate page in the
65+
// future?
66+
.filter(|team| {
67+
matches!(
68+
team.subteam_of.as_deref(),
69+
None | Some("launching-pad") | Some("leadership-council")
70+
)
71+
})
6372
.map(|team| IndexTeam {
6473
url: format!(
6574
"{}/{}",
@@ -99,9 +108,11 @@ impl Data {
99108

100109
// Don't show pages for subteams
101110
if let Some(subteam) = &main_team.subteam_of {
102-
// Launching-pad does not have a page of its own, but we do want
103-
// to show the working groups that are inside it.
104-
if subteam != "launching-pad" {
111+
// Each launching-pad and leadership-council subteam has their own
112+
// page. Subteams of those subteams do not get a page of their own
113+
// (they are shown in their parent page). We may want to consider
114+
// putting launching-pad teams into a separate page in the future.
115+
if !matches!(subteam.as_ref(), "launching-pad" | "leadership-council") {
105116
return Err(TeamNotFound.into());
106117
}
107118
}
@@ -120,6 +131,9 @@ impl Data {
120131

121132
self.teams
122133
.into_iter()
134+
// The leadership-council page should show just the
135+
// leadership-council, not everything underneath it.
136+
.filter(|_| main_team.name != "leadership-council")
123137
.filter(|team| team.website_data.is_some())
124138
.filter(|team| {
125139
// For teams find not only direct subteams but also transitive ones.

templates/governance/group-team.html.hbs

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
<a id="{{team-text team name}}" name="{{team.name}}"></a>
1+
<a id="team-{{team.name}}"></a>
22
<div class="w-100 mw-none mw-8-m mw9-l ph3 center f3">
33
<header class="pb0 pt5">
4-
<a class="linkable-subheading" href="#{{team-text team name}}"><h2>{{team-text team name}}</h2></a>
4+
<a class="linkable-subheading" href="#team-{{team.name}}"><h2>{{team-text team name}}</h2></a>
55
<div class="highlight"></div>
66
</header>
77
<div>
@@ -36,6 +36,13 @@
3636
</a>
3737
</div>
3838
{{/if}}
39+
{{#if team.website_data.matrix_room}}
40+
<div class="pt4 pt3-l flex flex-column flex-row-l items-center-l">
41+
<a class="button button-secondary" href="https://matrix.to/#/{{team.website_data.matrix_room}}">
42+
{{#fluent "governance-team-matrix-room"}}{{#fluentparam "room"}}{{team.website_data.matrix_room}}{{/fluentparam}}{{/fluent}}
43+
</a>
44+
</div>
45+
{{/if}}
3946
</div>
4047
</div>
4148
</div>

0 commit comments

Comments
 (0)