Skip to content

feat(ForcedDecisions): add forced-decisions APIs to OptimizelyUserContext #233

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 28 commits into from
Nov 4, 2021
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
8607ff2
Work in progress
NomanShoaib Sep 21, 2021
e013fea
fix
NomanShoaib Sep 21, 2021
0b1f517
implemented forcedDecision
NomanShoaib Sep 22, 2021
cd4176c
fixed issues
NomanShoaib Sep 24, 2021
5c753aa
fixed all errors
NomanShoaib Sep 27, 2021
0fd717b
fixed old tests and logs errors
NomanShoaib Sep 27, 2021
7e2400f
fix
NomanShoaib Sep 28, 2021
9cb1d03
fix
NomanShoaib Sep 28, 2021
ae48a61
added remove forcedecision removeall force decision apis
NomanShoaib Sep 28, 2021
6a9a5f0
Unit tests added and fixed some bugs
NomanShoaib Oct 1, 2021
60ae438
Added unit tests of forceddecision
NomanShoaib Oct 4, 2021
df1c82f
Comments fixed
NomanShoaib Oct 6, 2021
80e5026
return true in removeallforcedecision even if forceddecision list is …
NomanShoaib Oct 6, 2021
979a0e7
Added depreciation warning in
NomanShoaib Oct 6, 2021
d22ad11
Fixed experimentId null or empty error in impression event
NomanShoaib Oct 6, 2021
a5843c2
fixed logging and !=null comments
NomanShoaib Oct 6, 2021
f387f89
lint fix
NomanShoaib Oct 6, 2021
3557793
removed depreciation warning
NomanShoaib Oct 7, 2021
8797195
made rule key compulsory instead of optional and moved it to 2nd arg …
NomanShoaib Oct 11, 2021
83ef326
- moved the two methods of remove and remove all forceddecision after…
NomanShoaib Oct 11, 2021
b7b980d
Added Decided by forceddecision reason
NomanShoaib Oct 12, 2021
280371c
Added reasons in test case
NomanShoaib Oct 13, 2021
9168430
DatafileprojectConfig line 248 added verification that redundant vari…
NomanShoaib Oct 14, 2021
2bb8937
fixed logging and unit tests
NomanShoaib Oct 15, 2021
0df9732
lint fix
NomanShoaib Oct 20, 2021
bc3bb03
OptimizelyDecisionContext and OptimizelyForcedDecision implemented
ozayr-zaviar Oct 29, 2021
7566d55
impression corrected (#234)
ozayr-zaviar Nov 3, 2021
32122b7
comments addressed
ozayr-zaviar Nov 4, 2021
File filter

Filter by extension

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
lint fix
  • Loading branch information
NomanShoaib committed Oct 20, 2021
commit 0df97322bad12ecbef7e6512a1b7fda59831faef
1 change: 0 additions & 1 deletion tests/OptimizelyTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -5285,7 +5285,6 @@ public function testGetEnabledFeaturesCallsDecisionListenerForAllFeatures()
$decision8,
$decision9,
$decision10

));

$optimizelyMock->notificationCenter = $this->notificationCenterMock;
Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy