`future.update_flag` should not require explicit priority

Offen
#8,656 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
70/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
api, backend

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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 priority field is optional when passing segment overrides to the update_flag service, or to the update-flag endpoint, as consequence.
  • When omitted, the priority of a segment override becomes 0, while the priority of other overrides for the same feature is shifted down.
    • This keeps the behaviour we implemented in flag dependency.
Vorherrschende Sprache
Python
Sterne
7.1k
Forks
603
Ø Merge
3 T. 8 Std.
Gemergte PRs (30 T.)
102

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Flagsmith/flagsmith

Alle Issues in Flagsmith/flagsmith

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.