Skip to content

Commit 3ba4770

Browse files
authored
feat(front): add organization_id to workflow query params (#600)
## 📝 Description This change ensures that `organization_id` is included when listing workflows. By providing `organization_id` the backend can leverage more selective indexes, which should improve query planning and performance. See [related task](renderedtext/tasks#8617). ## ✅ Checklist - [x] I have tested this change - [x] ~This change requires documentation update~ N/A
1 parent 65fe8a5 commit 3ba4770

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

front/lib/front_web/controllers/dashboard_controller.ex

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ defmodule FrontWeb.DashboardController do
7777
end
7878
end
7979

80-
defp params(conn, page_token, direction, filters, project_ids) do
80+
defp params(conn, page_token, direction, filters, project_ids, org_id) do
8181
user_id = conn.assigns.user_id
8282
direction = map_workflow_direction(direction)
8383

@@ -90,7 +90,8 @@ defmodule FrontWeb.DashboardController do
9090
page_token: page_token,
9191
direction: direction,
9292
created_after: time,
93-
project_ids: project_ids
93+
project_ids: project_ids,
94+
organization_id: org_id
9495
]
9596
|> inject_requester(filters.requester, user_id)
9697
end
@@ -118,7 +119,7 @@ defmodule FrontWeb.DashboardController do
118119
{:ok, project_ids} = Front.RBAC.Members.list_accessible_projects(org_id, user_id)
119120

120121
filters = %{requester: requester}
121-
params = params(conn, page_token, direction, filters, project_ids)
122+
params = params(conn, page_token, direction, filters, project_ids, org_id)
122123

123124
{workflows, next_page_token, previous_page_token} = list_workflows(params)
124125

@@ -140,7 +141,8 @@ defmodule FrontWeb.DashboardController do
140141
params: [
141142
page_token: page_token,
142143
direction: direction,
143-
requester: requester
144+
requester: requester,
145+
organization_id: org_id
144146
]
145147
}
146148

@@ -525,7 +527,7 @@ defmodule FrontWeb.DashboardController do
525527
Front.RBAC.Members.list_accessible_projects(org_id, user_id)
526528
end)
527529

528-
params = params(conn, page_token, direction, filters, project_ids)
530+
params = params(conn, page_token, direction, filters, project_ids, org_id)
529531

530532
{workflows, next_page_token, previous_page_token} =
531533
Watchman.benchmark("home_page.list_workflows", fn ->
@@ -550,7 +552,8 @@ defmodule FrontWeb.DashboardController do
550552
params: [
551553
page_token: page_token,
552554
direction: direction,
553-
requester: filters.requester
555+
requester: filters.requester,
556+
organization_id: org_id
554557
]
555558
}
556559

0 commit comments

Comments
 (0)