Duplicate usernames in combo-box when filtering patches by delegate
還沒有人認領這個 Issue。
評估
研究方向
從 patchwork/filters.py 中 DelegateFilter 附近的第 458 行開始,檢查用於專案範圍 delegate 查詢的 _get_delegate_qs。確認表單如何接收目前的專案。完成標準是:組合方塊列出所選專案的每位 maintainer 且各出現一次,並排除其他專案的 maintainer。
由索引模型根據 Issue 內容生成。
描述
The combo-box with the list of delegates to filter by has duplicate usernames. It lists a maintainer more than once if he/she maintains multiple projects. It also lists maintainers of other projects who don't maintain the one I'm filtering patches in.
This happens because DelegateFilter's form doesn't filter by project:
https://github.com/getpatchwork/patchwork/blame/bdb049c7939b7cdea1eddf029a0bc47cb338da67/patchwork/filters.py#L458
delegates = User.objects.filter(
profile__maintainer_projects__isnull=False)
Eliminating duplicate rows by using distinct() works, but we still have the issue of different project maintainers being listed in the same list:
delegates = User.objects.filter(
profile__maintainer_projects__isnull=False).distinct()
Maybe we should use _get_delegate_qs instead?
- 主要語言
- Python
- 星號
- 317
- 分支
- 91
- PR 合併指標
- 30 天內沒有已合併 PR
貢獻指南
從這裡開始
- 先讀完整個 Issue,再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫,在一個分支上完成修改。
- 送出 Pull Request,並在描述裡引用這個 Issue 編號。
getpatchwork/patchwork 的其他 Issue
-
難度 2/5 1-3 小時 新手友好度 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
難度 2/5 1-3 小時 新手友好度 74/100
getpatchwork/patchwork#546 · 2 則留言 ·
-
難度 3/5 1-2 天 新手友好度 52/100
getpatchwork/patchwork#664 ·
-
難度 4/5 3-5 天 新手友好度 48/100
getpatchwork/patchwork#663 ·
-
難度 3/5 1-2 天 新手友好度 45/100
getpatchwork/patchwork#645 ·
查看 getpatchwork/patchwork 的全部 Issue
相似的 Issue
-
agent-ready documentation needs-triage
難度 1/5 1-3 小時 新手友好度 88/100
-
documentation
難度 1/5 1 小時以內 新手友好度 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" 未關閉
難度 1/5 1 小時以內 新手友好度 92/100
-
instance instance add
難度 1/5 1 小時以內 新手友好度 72/100
searxng/searx-instances#939 · 1 則留言 ·
-
area-deployment area-integrations triage:bot-seen
難度 2/5 半天 新手友好度 86/100