Skip to content

Conversation

@sdzh-atlassian
Copy link
Member

What Is This Change?

Note: no change in behaviour or presentation, refactoring only

Another step in centralizing our process.env usage - moved the feature flag override logic into static config, and the parsing - to a separate file. Added extra tests and documentation

There are now 3 ways to add an override:

  • In code, if you need a fast & temporary update.
  • In .env, if you'd like to persist the change in your local workspace* - expected format is documented
  • In the actual FF service

* - the .env update currently requires a full re-build, will fix this in another PR

This specifically relates to overrides, so normal FF client logic isn't affected

How Has This Been Tested?

See demo:
https://www.loom.com/share/5cfdbe8832aa4584a57ac0c81d25610f

Basic checks:

  • npm run lint
  • npm run test

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

Successfully merging this pull request may close these issues.

2 participants