Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cozy-flags] flags may be undefined if set outside a component #2550

Open
JF-Cozy opened this issue Jun 6, 2024 · 2 comments
Open

[cozy-flags] flags may be undefined if set outside a component #2550

JF-Cozy opened this issue Jun 6, 2024 · 2 comments

Comments

@JF-Cozy
Copy link
Contributor

JF-Cozy commented Jun 6, 2024

Si le flag est définit directement dans un fichier js et utilisé ensuite dans un composant, il se peut que la valeur soit d'abord undefined avant d'être la bonne. Le code est exécuté directement par le browser dès qu'il le lit, et sûrement avant que le plugin de flag ce soit initialisé.

const myflag = flag('myflag') // risque que le flag soit à undefined alors qu'il a une valeur

const MyComponent = () => {
  const myflag = flag('myflag') // là ça devrait être bon

  if (myflag) {
    return ...
  } else {
    return ...
  }
}
@Crash--
Copy link
Contributor

Crash-- commented Jun 6, 2024

On avait déjà eu des soucis similaires cf #2305 et il me semblait qu'on avait déjà fait passer le mot.

Mais ça semble pas suffisant, quelles solutions possibles ?

@zatteo
Copy link
Contributor

zatteo commented Jun 6, 2024

Mais ça semble pas suffisant, quelles solutions possibles ?

On peut peut-être déclencher un warning côté cozy-flags si on essaie de récupérer un flag alors que le store de flag n'a pas été initialisé ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants