Skip to content

[FSSDK-11181] chore: add cmab uuid into impression event meta #603

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

Jump to bottom
Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

[FSSDK-11181] chore: add cmab uuid into impression event meta #603

wants to merge 27 commits into from

Conversation

muzahidul-opti
Copy link
Contributor

@muzahidul-opti muzahidul-opti commented Jul 7, 2025 •
edited
Loading

Summary

  • Add cmab uuid into event meta

Test plan

  • All existing test cases should be passed

Issues

  • FSSDK-11181

@muzahidul-opti
wip: cmab client done
b40adda
@muzahidul-opti
Update test cases
9c9b9b0
@muzahidul-opti
wip: add test cases for cmab services
d998e0b
@muzahidul-opti
Merge branch 'master' into muzahid/cmab-service
c427a46 
# Conflicts:
#	OptimizelySwiftSDK.xcodeproj/project.pbxproj
#	Sources/CMAB/CmabClient.swift
#	Tests/OptimizelyTests-Common/CMABClientTests.swift
@muzahidul-opti
Added log for cmab decision
8e168f6
@muzahidul-opti
Update copyright date
0c4d072
@muzahidul-opti
CMAB decision implemented
f342ae2
@muzahidul-opti
CmabService testcases added for sync getDecision function
ebbde3c
@muzahidul-opti
Add factory method for DefaultCmabService
92d3140
@muzahidul-opti
DefaultDecision initializer updated
c6507b0
@muzahidul-opti
Add operation type enum
b51c228
@muzahidul-opti
CMAB not supported in sync mode
e9d01ac
@muzahidul-opti
Reuse experiment bucketing logic
e827b27
@muzahidul-opti
Add seperate method for group exlusion, add bucketToEntityId method
445006b
@muzahidul-opti
Update bucketing logic for cmab experiment
b9ce0f5
@muzahidul-opti
Add test cases for cmab experiement
093e372
@muzahidul-opti
Add test cases for cmab decision options
f2cc7df
@muzahidul-opti
Merge branch 'master' into muzahid/cmab-decision
f0c1d5b 
# Conflicts:
#	Sources/CMAB/CmabService.swift
#	Tests/OptimizelyTests-Common/CmabServiceTests.swift
@muzahidul-opti
Add test cases for bucketToEntity
384bfa8
@muzahidul-opti
Return feature dicision with nil variation for CMAB fetch error
87bcecf
@muzahidul-opti
Add code doc and fix linting issue
d1067d6
@muzahidul-opti
Update cmab entity id matching logic
0d761c9
@muzahidul-opti
Add cmab uuid to metadata
bd2a47c
@muzahidul-opti
Add test for impression event
eff199b
@muzahidul-opti muzahidul-opti marked this pull request as ready for review July 14, 2025 11:41
@muzahidul-opti muzahidul-opti requested a review from raju-opti July 14, 2025 12:45
@muzahidul-opti muzahidul-opti requested a review from jaeopt July 14, 2025 12:45
jaeopt
jaeopt approved these changes Jul 14, 2025
View reviewed changes
Copy link
Contributor

@jaeopt jaeopt left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaeopt jaeopt jaeopt approved these changes

@raju-opti raju-opti Awaiting requested review from raju-opti

Assignees

@muzahidul-opti muzahidul-opti

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@muzahidul-opti @jaeopt