Skip to content

feat: Add ODP Segments support in Audience Evaluation #474

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.

Sign up for GitHub

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 28 commits into from
Jul 21, 2022
Merged

feat: Add ODP Segments support in Audience Evaluation #474

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
2ba5621
OASIS-8309 Added qualified segments to User Context, updated evaluate…
genemitelmanopti May 23, 2022
6d1924a
OASIS-8309 Updated UserAttribute evaluate to check qualified segments.
genemitelmanopti May 24, 2022
0d9d3b8
Added unit tests and fixed issues
NomanShoaib Jun 16, 2022
a8d0c3f
Updated header
NomanShoaib Jun 16, 2022
b461f82
resolved comments
NomanShoaib Jun 21, 2022
8ccd995
Resolved spotbugs error
NomanShoaib Jun 21, 2022
f449dda
removed unnecessary null check
NomanShoaib Jun 21, 2022
a9c8d05
Merge branch 'master' into gene/ats
mnoman09 Jun 27, 2022
315a9c9
Comment added
NomanShoaib Jun 28, 2022
0469623
1. Implemented parsing of integrations in all four Datafile Parsers.
zashraf1985 Jul 16, 2022
15bfe93
Update core-api/src/main/java/com/optimizely/ab/OptimizelyUserContext…
mnoman09 Jul 18, 2022
b8eaae1
Resolved comment and fixed
NomanShoaib Jul 18, 2022
eb2e8c8
trying to fix a test
zashraf1985 Jul 18, 2022
1ba1b3e
trying ignoing some tests
zashraf1985 Jul 18, 2022
9910be8
IGNORED the correct test
zashraf1985 Jul 18, 2022
6af28aa
Merge branch 'master' into gene/ats
msohailhussain Jul 18, 2022
226d3ae
Enabled ignored tests
zashraf1985 Jul 19, 2022
24396e9
test
mnoman09 Jul 19, 2022
12d92ee
test
mnoman09 Jul 19, 2022
889e405
Reverting assertEqual to assertThat
mnoman09 Jul 19, 2022
29750c4
test
mnoman09 Jul 19, 2022
cc4ee31
test
mnoman09 Jul 19, 2022
4f61e54
using macos as in git action there is some problem with ubuntu 18.04
mnoman09 Jul 19, 2022
99a7f0a
change os version to macos latest for unit test
mnoman09 Jul 19, 2022
aad5d0b
added integrations to parsing tests
zashraf1985 Jul 19, 2022
c0f0284
Incorporate review feedback
zashraf1985 Jul 20, 2022
d6ec898
Added unit tests for integrations parsing
zashraf1985 Jul 21, 2022
5df10d4
Added some more unit tests
zashraf1985 Jul 21, 2022
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
removed unnecessary null check
  • Loading branch information
@NomanShoaib
NomanShoaib committed Jun 21, 2022
commit f449ddadb69bc83e6e68788ad5fd8612d54cc1df
3 changes: 0 additions & 3 deletions core-api/src/main/java/com/optimizely/ab/config/audience/UserAttribute.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,6 @@ public Object getValue() {
@Nullable
public Boolean evaluate(ProjectConfig config, OptimizelyUserContext user) {
Map<String,Object> attributes = user.getAttributes();
if (attributes == null) {
attributes = Collections.emptyMap();
}
// Valid for primitive types, but needs to change when a value is an object or an array
Object userAttributeValue = attributes.get(name);

Expand Down