Jonathan Norris

Results 17 comments of Jonathan Norris

For context, this method was exported when we were previously using the ESM bindings, but do a whole host of issues with the support for ESM we decided to switch...

Current dependencies on/for this PR: * main * **PR #176** * **PR #177** * **PR #178** 👈 This comment was auto-generated by [Graphite](https://app.graphite.dev/github/pr/DevCycleHQ/ios-client-sdk/178?utm_source=stack-comment).

Hey @maxveldink, what's the motivation behind having the `type` in the data model? I think we might have a better chance of constraining the usage of this to its intended...

> ## This PR > Adds an optional `allFlagMetadata` accessor to clients and the provider interface for retrieving a list of flag keys available in the provider. > > *...

@ajwootto can we merge this?

* **#840** 👈 * **#838** * `main` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @jonathannorris and the rest of your teammates on Graphite

* **#840** * **#838** 👈 * `main` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @jonathannorris and the rest of your teammates on Graphite

Current dependencies on/for this PR: * main * **PR #560** * **PR #561** * **PR #564** * **PR #565** * **PR #566** 👈 This comment was auto-generated by [Graphite](https://app.graphite.dev/github/pr/DevCycleHQ/js-sdks/566?utm_source=stack-comment).

@ajwootto is this PR still valid?

ugh @jsalaber now that change means this is an issue: Which is probably a lot of folks are using `identifyUser` without setting a callback :(