Skip to content

Commit 8e996b0

Browse files
authored
Merge pull request #277 from qccoders/card-domain
Fix accepted card numbers
2 parents 3b43dc6 + 564f803 commit 8e996b0

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

api/QCVOC.Api/Veterans/Data/DTO/VeteranEnrollRequest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public class VeteranEnrollRequest
4242
/// <summary>
4343
/// Gets or sets the number of the card presently assigned to the Veteran.
4444
/// </summary>
45-
[Range(minimum: 1000, maximum: 9999)]
45+
[Range(minimum: 0, maximum: 9999)]
4646
public int? CardNumber { get; set; }
4747

4848
/// <summary>

api/QCVOC.Api/Veterans/Data/DTO/VeteranUpdateRequest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public class VeteranUpdateRequest
4242
/// <summary>
4343
/// Gets or sets the number of the card presently assigned to the Veteran.
4444
/// </summary>
45-
[Range(minimum: 1000, maximum: 9999)]
45+
[Range(minimum: 0, maximum: 9999)]
4646
public int? CardNumber { get; set; }
4747

4848
/// <summary>

web/src/veterans/VeteranDialog.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ class VeteranDialog extends Component {
231231
let { cardNumber, firstName, lastName, address, primaryPhone, email } = this.state.veteran;
232232
let result = { ...initialState.validation };
233233

234-
if (cardNumber !== '' && (isNaN(cardNumber) || cardNumber < 1000 || cardNumber > 9999)) result.cardNumber = 'The Card Number field must be a number between 1000 and 9999.';
234+
if (cardNumber !== '' && (isNaN(cardNumber) || cardNumber < 0 || cardNumber > 9999)) result.cardNumber = 'The Card Number field must be a number between 0 and 9999.';
235235
if (firstName === '') result.firstName = 'The First Name field is required.';
236236
if (lastName === '') result.lastName = 'The Last Name field is required.';
237237
if (address === '') result.address = 'The Address field is required.';

0 commit comments

Comments
 (0)