Skip to content

chore(optimizely-sdk): Prepare for 3.3.0-beta release #354

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 21, 2019

Conversation

mjc1283
Copy link
Contributor

@mjc1283 mjc1283 commented Aug 21, 2019

No description provided.

@mjc1283 mjc1283 requested review from mikeproeng37 and a team August 21, 2019 15:33
@mjc1283 mjc1283 removed the request for review from mikeproeng37 August 21, 2019 15:36
@mjc1283 mjc1283 removed their assignment Aug 21, 2019
@coveralls
Copy link

coveralls commented Aug 21, 2019

Coverage Status

Coverage remained the same at 97.598% when pulling 9683ee1 on mcarroll/prepare-3.3.0-beta into 12a4a51 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.598% when pulling b5b3fca on mcarroll/prepare-3.3.0-beta into 12a4a51 on master.

Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

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

Lgtm just one nit

### New Features

- Added support for event batching via the event processor.
- Events generated by methods like `activate`, `track`, and `isFeatureEnabled` will be held in a queue until the configured batch size is reached, or the configured flush interval has elapsed. Then, they will be combined into a request and send to the event dispatcher.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: typo toward the end.. "sent"

@mjc1283 mjc1283 merged commit f71646a into master Aug 21, 2019
@mjc1283 mjc1283 deleted the mcarroll/prepare-3.3.0-beta branch August 21, 2019 16:26
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.

3 participants