Skip to content

Commit c590ac8

Browse files
authored
Remove tribe field from payment account create (#655)
* Removed the range validation attribute from the FxDestinationAmount. (#654) * Remove redundant Tribe field from create payment account, was causing confusion.
1 parent e60d024 commit c590ac8

File tree

3 files changed

+2
-10
lines changed

3 files changed

+2
-10
lines changed

src/NoFrixion.MoneyMoov/Models/Account/PaymentAccountCreate.cs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,15 +58,10 @@ public class PaymentAccountCreate
5858
/// </summary>
5959
public AccountTypeEnum? AccountType { get; set; }
6060

61-
/// <summary>
62-
/// If creating a Tribe account type, then this is the tribe account id
63-
/// </summary>
64-
public string? TribeAccountId { get; set; }
65-
6661
/// <summary>
6762
/// For EUR accounts this can be set to the ID of another account that will act as the
68-
/// backing phyiscal account. The new account will then act as a Virtual account, able to
69-
/// receive funds but the transactions will be recorded aginst the backing physical account.
63+
/// backing physical account. The new account will then act as a Virtual account, able to
64+
/// receive funds but the transactions will be recorded against the backing physical account.
7065
/// </summary>
7166
public Guid? PhysicalAccountID { get; set; }
7267

@@ -98,7 +93,6 @@ public Dictionary<string, string> ToDictionary()
9893
{ nameof(AccountName), AccountName ?? string.Empty },
9994
{ nameof(SupplierPhysicalAccountID), SupplierPhysicalAccountID.ToString() },
10095
{ nameof(AccountType), AccountType?.ToString() ?? string.Empty },
101-
{ nameof(TribeAccountId), TribeAccountId ?? string.Empty },
10296
{ nameof(PhysicalAccountID), PhysicalAccountID?.ToString() ?? string.Empty },
10397
{nameof(IsTrustAccount), IsTrustAccount.ToString()}
10498
};

src/NoFrixion.MoneyMoov/Models/Payouts/PayoutCreate.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,6 @@ public Guid? BeneficiaryID
240240
/// Optional but one of Amount or FxDestinationAmount must be set. If specified this will be the amount sent to the payee.
241241
/// The payout's Amount will be dynamically adjusted based on this amount and the FX rate.
242242
/// </summary>
243-
[Range(1.00, double.MaxValue, ErrorMessage = "Minimum value of 1.00 is required for FxDestinationAmount.")]
244243
public decimal? FxDestinationAmount { get; set; }
245244

246245
/// <summary>

src/NoFrixion.MoneyMoov/Models/Payouts/PayoutUpdate.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ public Guid? BeneficiaryID
216216
/// Optional but one of Amount or FxDestinationAmount must be set. If specified this will be the amount sent to the payee.
217217
/// The payout's Amount will be dynamically adjusted based on this amount and the FX rate.
218218
/// </summary>
219-
[Range(1.00, double.MaxValue, ErrorMessage = "Minimum value of 1.00 is required for FxDestinationAmount.")]
220219
public decimal? FxDestinationAmount { get; set; }
221220

222221
/// <summary>

0 commit comments

Comments
 (0)