Skip to content

Conversation

thomaszurkan-optimizely
Copy link
Contributor

No description provided.

CHANGELOG.md Outdated

January 30, 2018

This release adds support for bucketing id (By passing in `$opt_bucketing_id` in the attribute map to override the user id as the bucketing variable. This is useful when wanting a set of users to share the same experience such as two players i n a game). This release also depricates the old notification broadcaster in favor of a notification center that supports a wide range of notifications. The notification center is also consistently implemented in all Optimizely SDKs.
Copy link
Contributor

Choose a reason for hiding this comment

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

remove the space between i and n at "players i n a game".

CHANGELOG.md Outdated
### New Features

- Added `$opt_bucketing_id` in the attribute map for overriding bucketing using the user id.
- Optimizely notificaiton center for activate and track notifiations.
Copy link
Contributor

Choose a reason for hiding this comment

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

give a more in depth description of what the notification center is and how it works.

CHANGELOG.md Outdated

### Breaking Changes

- Nothing breaking from 1.8.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Should remove this

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

@thomaszurkan-optimizely thomaszurkan-optimizely merged commit 52476f9 into master Jan 31, 2018
thomaszurkan-optimizely added a commit that referenced this pull request Jan 31, 2018
* update changelog

* address comments and expand on notification listener.
@thomaszurkan-optimizely thomaszurkan-optimizely deleted the update/changelog1.9.x branch January 31, 2018 19:50
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