Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit7b65422

Browse files
authored
fix: change notifications actions url (#17083)
Related to#17082Some notifications ( workspace created and workspace manually updated )are using wrong variables to build the Action URL. Fixing it.
1 parent081679f commit7b65422

9 files changed

+80
-28
lines changed
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
UPDATE notification_templates
2+
SET
3+
actions='[
4+
{
5+
"label": "View workspace",
6+
"url": "{{base_url}}/@{{.UserUsername}}/{{.Labels.workspace}}"
7+
}
8+
]'::jsonb
9+
WHERE id='281fdf73-c6d6-4cbb-8ff5-888baf8a2fff';
10+
11+
UPDATE notification_templates
12+
SET
13+
actions='[
14+
{
15+
"label": "View workspace",
16+
"url": "{{base_url}}/@{{.UserUsername}}/{{.Labels.workspace}}"
17+
},
18+
{
19+
"label": "View template version",
20+
"url": "{{base_url}}/templates/{{.Labels.organization}}/{{.Labels.template}}/versions/{{.Labels.version}}"
21+
}
22+
]'::jsonb
23+
WHERE id='d089fe7b-d5c5-4c0c-aaf5-689859f7d392';
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
UPDATE notification_templates
2+
SET
3+
actions='[
4+
{
5+
"label": "View workspace",
6+
"url": "{{base_url}}/@{{.Labels.workspace_owner_username}}/{{.Labels.workspace}}"
7+
}
8+
]'::jsonb
9+
WHERE id='281fdf73-c6d6-4cbb-8ff5-888baf8a2fff';
10+
11+
UPDATE notification_templates
12+
SET
13+
actions='[
14+
{
15+
"label": "View workspace",
16+
"url": "{{base_url}}/@{{.Labels.workspace_owner_username}}/{{.Labels.workspace}}"
17+
},
18+
{
19+
"label": "View template version",
20+
"url": "{{base_url}}/templates/{{.Labels.organization}}/{{.Labels.template}}/versions/{{.Labels.version}}"
21+
}
22+
]'::jsonb
23+
WHERE id='d089fe7b-d5c5-4c0c-aaf5-689859f7d392';

‎coderd/notifications/notifications_test.go‎

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1074,9 +1074,10 @@ func TestNotificationTemplates_Golden(t *testing.T) {
10741074
UserEmail:"bobby@coder.com",
10751075
UserUsername:"bobby",
10761076
Labels:map[string]string{
1077-
"workspace":"bobby-workspace",
1078-
"template":"bobby-template",
1079-
"version":"alpha",
1077+
"workspace":"bobby-workspace",
1078+
"template":"bobby-template",
1079+
"version":"alpha",
1080+
"workspace_owner_username":"mrbobby",
10801081
},
10811082
},
10821083
},
@@ -1088,11 +1089,12 @@ func TestNotificationTemplates_Golden(t *testing.T) {
10881089
UserEmail:"bobby@coder.com",
10891090
UserUsername:"bobby",
10901091
Labels:map[string]string{
1091-
"organization":"bobby-organization",
1092-
"initiator":"bobby",
1093-
"workspace":"bobby-workspace",
1094-
"template":"bobby-template",
1095-
"version":"alpha",
1092+
"organization":"bobby-organization",
1093+
"initiator":"bobby",
1094+
"workspace":"bobby-workspace",
1095+
"template":"bobby-template",
1096+
"version":"alpha",
1097+
"workspace_owner_username":"mrbobby",
10961098
},
10971099
},
10981100
},

‎coderd/notifications/testdata/rendered-templates/smtp/TemplateWorkspaceCreated.html.golden‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ The workspace bobby-workspace has been created from the template bobby-temp=
1616
late using version alpha.
1717

1818

19-
View workspace: http://test.com/@bobby/bobby-workspace
19+
View workspace: http://test.com/@mrbobby/bobby-workspace
2020

2121
--bbe61b741255b6098bb6b3c1f41b885773df633cb18d2a3002b68e4bc9c4
2222
Content-Transfer-Encoding: quoted-printable
@@ -53,9 +53,9 @@ ha</strong>.</p>
5353
</div>
5454
<div style=3D"text-align: center; margin-top: 32px;">
5555
=20
56-
<a href=3D"http://test.com/@bobby/bobby-workspace" style=3D"display=
57-
: inline-block; padding: 13px 24px; background-color: #020617; color: #f8fa=
58-
fc; text-decoration: none; border-radius: 8px; margin: 0 4px;">
56+
<a href=3D"http://test.com/@mrbobby/bobby-workspace" style=3D"displ=
57+
ay: inline-block; padding: 13px 24px; background-color: #020617; color: #f8=
58+
fafc; text-decoration: none; border-radius: 8px; margin: 0 4px;">
5959
View workspace
6060
</a>
6161
=20

‎coderd/notifications/testdata/rendered-templates/smtp/TemplateWorkspaceManuallyUpdated.html.golden‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ A new workspace build has been manually created for your workspace bobby-wo=
1616
rkspace by bobby to update it to version alpha of template bobby-template.
1717

1818

19-
View workspace: http://test.com/@bobby/bobby-workspace
19+
View workspace: http://test.com/@mrbobby/bobby-workspace
2020

2121
View template version: http://test.com/templates/bobby-organization/bobby-t=
2222
emplate/versions/alpha
@@ -57,9 +57,9 @@ g>.</p>
5757
</div>
5858
<div style=3D"text-align: center; margin-top: 32px;">
5959
=20
60-
<a href=3D"http://test.com/@bobby/bobby-workspace" style=3D"display=
61-
: inline-block; padding: 13px 24px; background-color: #020617; color: #f8fa=
62-
fc; text-decoration: none; border-radius: 8px; margin: 0 4px;">
60+
<a href=3D"http://test.com/@mrbobby/bobby-workspace" style=3D"displ=
61+
ay: inline-block; padding: 13px 24px; background-color: #020617; color: #f8=
62+
fafc; text-decoration: none; border-radius: 8px; margin: 0 4px;">
6363
View workspace
6464
</a>
6565
=20

‎coderd/notifications/testdata/rendered-templates/webhook/TemplateWorkspaceCreated.json.golden‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,14 @@
1212
"actions": [
1313
{
1414
"label": "View workspace",
15-
"url": "http://test.com/@bobby/bobby-workspace"
15+
"url": "http://test.com/@mrbobby/bobby-workspace"
1616
}
1717
],
1818
"labels": {
1919
"template": "bobby-template",
2020
"version": "alpha",
21-
"workspace": "bobby-workspace"
21+
"workspace": "bobby-workspace",
22+
"workspace_owner_username": "mrbobby"
2223
},
2324
"data": null,
2425
"targets": null

‎coderd/notifications/testdata/rendered-templates/webhook/TemplateWorkspaceManuallyUpdated.json.golden‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"actions": [
1313
{
1414
"label": "View workspace",
15-
"url": "http://test.com/@bobby/bobby-workspace"
15+
"url": "http://test.com/@mrbobby/bobby-workspace"
1616
},
1717
{
1818
"label": "View template version",
@@ -24,7 +24,8 @@
2424
"organization": "bobby-organization",
2525
"template": "bobby-template",
2626
"version": "alpha",
27-
"workspace": "bobby-workspace"
27+
"workspace": "bobby-workspace",
28+
"workspace_owner_username": "mrbobby"
2829
},
2930
"data": null,
3031
"targets": null

‎coderd/workspacebuilds.go‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -517,11 +517,12 @@ func (api *API) notifyWorkspaceUpdated(
517517
receiverID,
518518
notifications.TemplateWorkspaceManuallyUpdated,
519519
map[string]string{
520-
"organization":template.OrganizationName,
521-
"initiator":initiator.Name,
522-
"workspace":workspace.Name,
523-
"template":template.Name,
524-
"version":version.Name,
520+
"organization":template.OrganizationName,
521+
"initiator":initiator.Name,
522+
"workspace":workspace.Name,
523+
"template":template.Name,
524+
"version":version.Name,
525+
"workspace_owner_username":owner.Username,
525526
},
526527
map[string]any{
527528
"workspace":map[string]any{"id":workspace.ID,"name":workspace.Name},

‎coderd/workspaces.go‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -801,9 +801,10 @@ func (api *API) notifyWorkspaceCreated(
801801
receiverID,
802802
notifications.TemplateWorkspaceCreated,
803803
map[string]string{
804-
"workspace":workspace.Name,
805-
"template":template.Name,
806-
"version":version.Name,
804+
"workspace":workspace.Name,
805+
"template":template.Name,
806+
"version":version.Name,
807+
"workspace_owner_username":owner.Username,
807808
},
808809
map[string]any{
809810
"workspace":map[string]any{"id":workspace.ID,"name":workspace.Name},

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp