Skip to content

Bring develop into main for docs changes to be applied#331

Merged
louiszawadzki merged 20 commits into
mainfrom
develop
Nov 9, 2022
Merged

Bring develop into main for docs changes to be applied#331
louiszawadzki merged 20 commits into
mainfrom
develop

Conversation

@louiszawadzki

Copy link
Copy Markdown
Contributor

What does this PR do?

Bring develop into main for docs changes

louiszawadzki and others added 16 commits October 17, 2022 09:49
…le-rate-in-rule-psr

Add sample rate in rule psr
Co-authored-by: Heston Hoffman <hestonhoffman@gmail.com>
Good morning !

Thank you for providing this expo plugin (hey @louiszawadzki 😄 - hope you are doing well ! 🙂 )

I noticed this small issue with the doc, I think this extra key is needed to the config options if I'm not missing anything !
Docs: add needed key in expo plugin options doc
…es-docs-for-rn-70

Fix manual source map upload with hermes for RN over 0.70
@louiszawadzki louiszawadzki requested review from a team as code owners November 4, 2022 10:03
louiszawadzki and others added 4 commits November 8, 2022 10:46
Co-authored-by: Kari Halsted <12926135+kayayarai@users.noreply.github.com>
…-hybrid-apps

Add documentation on Hybrid applications
@louiszawadzki louiszawadzki merged commit 4f674f2 into main Nov 9, 2022
btthomas added a commit that referenced this pull request Jul 8, 2026
…PR1 review)

Depend on @datadog/flagging-core (and @openfeature/core, which its published types reference) and reuse its canonical wire types and configurationFromString instead of maintaining our own copies. Our local type names are kept as aliases so the rest of the SDK is insulated from upstream naming.

configurationToString stays local for now, with a loud TODO to adopt flagging-core's once the next major (>= 2.0.0) ships the fix from openfeature-js-client PR #331 (its 1.2.x serializer is broken). Consequences of delegating to flagging-core's parser: the bespoke parse-failure InternalLog warning is dropped (its parser swallows errors), and obfuscated is read via a small cast since it is absent from flagging-core's response type. integer/float variation types are kept as defensive runtime handling even though the CDN and flagging-core model only boolean/string/number/object.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

5 participants