Skip to content

Commit 70f2ec6

Browse files
authored
Merge pull request #180 from Riskified/DEV-114573
DEV-114573
2 parents ccb351d + b168657 commit 70f2ec6

File tree

8 files changed

+59
-8
lines changed

8 files changed

+59
-8
lines changed

riskified-sdk/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<modelVersion>4.0.0</modelVersion>
55
<groupId>com.riskified</groupId>
66
<artifactId>riskified-sdk</artifactId>
7-
<version>v3.1.0</version>
7+
<version>v3.2.0</version>
88
<name>Riskified SDK</name>
99
<description>Riskified rest api SDK for java</description>
1010
<url>https://www.riskified.com</url>

riskified-sdk/src/main/java/com/riskified/RiskifiedClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1075,7 +1075,7 @@ private HttpPost createPostRequest(String url) {
10751075
postRequest.setHeader(HttpHeaders.ACCEPT, "application/vnd.riskified.com; version=2");
10761076
postRequest.setHeader(HttpHeaders.ACCEPT, "application/json");
10771077
postRequest.setHeader("X-RISKIFIED-SHOP-DOMAIN", shopUrl);
1078-
postRequest.setHeader("User-Agent","riskified_java_sdk/3.1.0"); // TODO: take the version automatically
1078+
postRequest.setHeader("User-Agent","riskified_java_sdk/3.2.0"); // TODO: take the version automatically
10791079
postRequest.setHeader("Version",versionHeaderValue);
10801080
return postRequest;
10811081
}

riskified-sdk/src/main/java/com/riskified/models/BaseOrder.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,15 @@ public void validate(Validation validationType) throws FieldBadFormatException {
8383
Validate.notNull(this, this.createdAt, "Created At");
8484
Validate.notNull(this, this.updatedAt, "Updated At");
8585
Validate.notNullOrEmpty(this, this.gateway, "Gateway");
86-
Validate.notNullOrEmpty(this, this.browserIp, "Browser IP");
86+
if (this.source != null && !this.source.equalsIgnoreCase("phone")){
87+
Validate.notNullOrEmpty(this, this.browserIp, "Browser IP");
88+
}
89+
8790
Validate.notNull(this, this.totalPrice, "Total Price");
8891
Validate.notNull(this, this.lineItems, "Line Items");
89-
Validate.notNull(this, this.paymentDetails, "Payment Details");
92+
if (this.gateway != null && !this.gateway.equalsIgnoreCase("giftcard")){
93+
Validate.notNull(this, this.paymentDetails, "Payment Details");
94+
}
9095
Validate.notNull(this, this.customer, "Customer");
9196
Validate.notNull(this, this.billingAddress, "Billing Address");
9297
}

riskified-sdk/src/main/java/com/riskified/models/Customer.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,12 +62,14 @@ public Customer(String email, String firstName, String lastName, String id, Date
6262
public void validate(Validation validationType) throws FieldBadFormatException {
6363

6464
if (validationType == Validation.ALL) {
65-
Validate.notNullOrEmpty(this, this.email, "Email");
6665
Validate.notNullOrEmpty(this, this.firstName, "First Name");
6766
Validate.notNullOrEmpty(this, this.lastName, "Last Name");
68-
Validate.notNullOrEmpty(this, this.id, "Id");
69-
Validate.notNull(this, this.createdAt, "Created At");
70-
Validate.notNull(this, this.verifiedEmail, "Verified Email");
67+
if (this.accountType != null && !this.accountType.equalsIgnoreCase("guest")){
68+
Validate.notNullOrEmpty(this, this.email, "Email");
69+
Validate.notNullOrEmpty(this, this.id, "Id");
70+
Validate.notNull(this, this.createdAt, "Created At");
71+
Validate.notNull(this, this.verifiedEmail, "Verified Email");
72+
}
7173
}
7274

7375
if (this.email != null) {
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package com.riskified.models;
2+
3+
import java.util.List;
4+
5+
public class PolicyProtect {
6+
private List<UseCase> useCases;
7+
public List<UseCase> getUseCases() {
8+
return useCases;
9+
}
10+
public void setUseCases(List<UseCase> useCases) {
11+
this.useCases = useCases;
12+
}
13+
}

riskified-sdk/src/main/java/com/riskified/models/ResOrder.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ public class ResOrder {
1414
private Advice advice;
1515
private RecoveryEligibility recoveryEligibility;
1616
private Double riskScore;
17+
private PolicyProtect policyProtect;
1718

1819

1920
public String getId() {
@@ -105,4 +106,11 @@ public void setAction(String action) {
105106

106107
public Double getRiskScore(){return riskScore;}
107108

109+
public PolicyProtect getPolicyProtect() {
110+
return policyProtect;
111+
}
112+
public void setPolicyProtect(PolicyProtect policyProtect) {
113+
this.policyProtect = policyProtect;
114+
}
115+
108116
}
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
package com.riskified.models;
2+
3+
public class UseCase {
4+
private String useCase;
5+
private String decision;
6+
7+
public String getUseCase() {
8+
return useCase;
9+
}
10+
public void setUseCase(String useCase) {
11+
this.useCase = useCase;
12+
}
13+
public String getDecision() {
14+
return decision;
15+
}
16+
}

riskified-sdk/src/main/java/com/riskified/notifications/Notification.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.riskified.models.Context;
44
import com.riskified.models.Custom;
5+
import com.riskified.models.PolicyProtect;
56
import com.riskified.models.RecoveryEligibility;
67

78
// Shop URL may also be added to the API notifications from Riskified depending on your
@@ -32,6 +33,7 @@ public static class NotificationOrder {
3233
private RecoveryEligibility recoveryEligibility;
3334
private Context context;
3435
private String[] decision_reasons;
36+
private PolicyProtect policyProtect;
3537

3638
public String getId() {
3739
return id;
@@ -106,5 +108,10 @@ public String[] getDecisionReasons() {
106108
public void setDecisionReasons(String[] decisionReasons) {
107109
this.decision_reasons = decisionReasons;
108110
}
111+
112+
public PolicyProtect getPolicyProtect() {
113+
return policyProtect;
114+
}
115+
public void setPolicyProtect(PolicyProtect policyProtect) {}
109116
}
110117
}

0 commit comments

Comments
 (0)