`future.update_flag` should not require explicit priority

Ouverte
#8,656 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
70/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Active
Stack technique
python
Domaine
api, backend

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.
Langage dominant
Python
Étoiles
7.1k
Forks
603
Merge moyen
3 j 8 h
PR mergées (30 j)
102

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Flagsmith/flagsmith

Toutes les issues de Flagsmith/flagsmith

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.