`future.update_flag` should not require explicit priority
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start with the update_flag service and the /api/future/update-flag/ endpoint mentioned in the issue, then trace how segment overrides are handled. Done means priority can be omitted, defaults to 0, and other overrides for the same feature shift down in priority; verify the existing flag-dependency behavior remains intact.
索引モデルが issue の本文から書いたものです。
説明
This concerns the update_flag service, currently in use by the /api/__future__/update-flag/ endpoint, and also other new functionality throughout the API, such as flag dependencies.
While discussing how the update-flag endpoint should behave, we decided it should require explicit priority when setting a segment override. However, this creates the need of a ceremony when update_flag — the service — can be useful outside of the endpoint context.
Originally posted by @emyller in https://github.com/Flagsmith/flagsmith/pull/8632#discussion_r4158153920
Acceptance criteria
- The
priorityfield is optional when passing segment overrides to theupdate_flagservice, or to theupdate-flagendpoint, as consequence. - When omitted, the
priorityof a segment override becomes0, while the priority of other overrides for the same feature is shifted down.- This keeps the behaviour we implemented in flag dependency.
- 主要言語
- Python
- スター
- 7.1k
- フォーク
- 603
- 平均マージ
- 3日 8時間
- マージ済み PR（30日）
- 102
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Flagsmith/flagsmith のほかの issue
-
Expose whether an organisation is blocked for API usage対応中かも @PiUnknown が 26 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Flagsmith/flagsmith#8773 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
Environment colors mismatch in audit log view対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Flagsmith/flagsmith#8681 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
influx_query_manager doesn't catch ApiException, so Influx error responses 500対応中かも @SAY-5 が 1 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Flagsmith/flagsmith#8510 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Sales dashboard: download_org_data 500s with "Object of type generator is not JSON serializable"対応中かも @Iflaqbhat が 45 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
Flagsmith/flagsmith の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
peach-mitao/peach#693 ·
メンテナーはふだん 1 日以内に返信
-
FTP reads inside a transaction use a nonexistent staging path対応中かも @Harsh23Kashyap が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
fsspec/filesystem_spec#2463 ·
メンテナーはふだん 1 日以内に返信
-
comp/cli P2 tool/code-exec type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
NousResearch/hermes-agent#155989 ·
メンテナーはふだん 1 日以内に返信
-
PipesFileMessageReader parses a partially written message line: tail_file yields a line before its newline arrives対応中かも @PCTEJA が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
dagster-io/dagster#34442 ·
メンテナーはふだん 1 日以内に返信