Skip to content

Conversation

@mikeproeng37
Copy link
Contributor

@mikeproeng37 mikeproeng37 commented Jan 6, 2017

@optimizely/fullstack-devs

Team decided on a standardized demo app flow for the SDKs. We will expand on it later on with more complex scenarios and espresso (integration) tests. This work is tracked in the following ticket:

https://optimizely.atlassian.net/browse/OASIS-774

@wangjoshuah wangjoshuah self-requested a review January 6, 2017 02:16
Copy link
Contributor

@wangjoshuah wangjoshuah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

The SDK is implemented in the following way:
- The splash screen initializes the Optimizely manager asynchronously. This starts the datafile
fetch.
- Once the datafile is fetched and the Optimizely manager is started, we use grab the `optimizelyClient`
Copy link

@mauerbac mauerbac Jan 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...started, we grab the...

@mikeproeng37 mikeproeng37 merged commit 5bc91ac into devel Jan 6, 2017
@vraja2 vraja2 deleted the mng/new-demo-app branch January 10, 2017 18:47
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.

4 participants