`future.update_flag` should not require explicit priority
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 70/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 7.1k
- Fork
- 603
- Merge medio
- 3g 8h
- PR unite (30g)
- 102
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Flagsmith/flagsmith
-
Expose whether an organisation is blocked for API usageForse già presa @PiUnknown l’ha presa 26 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Flagsmith/flagsmith#8773 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Environment colors mismatch in audit log viewForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Flagsmith/flagsmith#8681 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
influx_query_manager doesn't catch ApiException, so Influx error responses 500Forse già presa @SAY-5 l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Flagsmith/flagsmith#8510 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Sales dashboard: download_org_data 500s with "Object of type generator is not JSON serializable"Forse già presa @Iflaqbhat l’ha presa 45 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di Flagsmith/flagsmith
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
peach-mitao/peach#693 ·
I maintainer di solito rispondono entro 1 giorno
-
FTP reads inside a transaction use a nonexistent staging pathForse già presa @Harsh23Kashyap l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
fsspec/filesystem_spec#2463 ·
I maintainer di solito rispondono entro 1 giorno
-
comp/cli P2 tool/code-exec type/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
NousResearch/hermes-agent#155989 ·
I maintainer di solito rispondono entro 1 giorno
-
PipesFileMessageReader parses a partially written message line: tail_file yields a line before its newline arrivesForse già presa @PCTEJA l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
dagster-io/dagster#34442 ·
I maintainer di solito rispondono entro 1 giorno