Skip to content

Conversation

@ioannisj
Copy link
Contributor

💡 Motivation and Context

Closes #367

Do not clear local flags on remote config error or when hasFeatureFlags is not false

💚 How did you test it?

Added a couple of unit tests

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

@ioannisj ioannisj requested a review from marandaneto as a code owner July 14, 2025 07:39
@ioannisj ioannisj changed the title feat: add unit tests feat: do not clear feature flags on remote config error Jul 14, 2025
@ioannisj ioannisj requested a review from a team July 14, 2025 07:42
@ioannisj ioannisj merged commit 5e062d2 into main Jul 14, 2025
10 checks passed
@ioannisj ioannisj deleted the fix/dont-clear-flags-on-config-error branch July 14, 2025 10:08
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.

Feature flags do not load from cache if app has network connectivity issues when opening

2 participants