Stop assuming all upperclassmen are on every packet
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 52/100
調査の方向性
packet/templates/upperclassmen_totals.html の39行目と、packet/routes/upperclassmen.py の51-62行目から始めます。upperclassmen のデータが active セクションに到達するまでを追跡し、そのうえで、表示される分母が各 upperclassman が署名できるパケットだけを数えるようにします。membership state によって以前のパケットが除外される場合に、active のカウントが num_total ではなく num_can_sign_as_upper を使用すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
This line makes the assumption that every single upperclassman is a required signature on every packet. This may not be the case, due to e.g. changes in membership state that don't influence earlier packets. This results in a confusing list of signature counts when an upperclassman is required on only some of the packets, since they are listed as $num_signed / $num_total in the active section. This aught to be $num_signed / $num_can_sign_as_upper
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/templates/upperclassmen_totals.html#L39
This will probably require injecting more data into the upperclassmen dict here so that it's accessible in the template.
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/routes/upperclassmen.py#L51-L62
- 主要言語
- Python
- スター
- 25
- フォーク
- 33
- 平均マージ
- 21日 1時間
- マージ済み PR（30日）
- 1
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ComputerScienceHouse/packet のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
ComputerScienceHouse/packet#413 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
ComputerScienceHouse/packet#411 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
ComputerScienceHouse/packet#408 ·
-
React Frontend? オープンux enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
ComputerScienceHouse/packet#404 ·
-
ux enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
ComputerScienceHouse/packet#403 ·
ComputerScienceHouse/packet の 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