Skip to content

Commit 0dd0fc0

Browse files
fix: add optional field
Signed-off-by: Axel Loupias <[email protected]>
1 parent 265e819 commit 0dd0fc0

File tree

7 files changed

+11
-10
lines changed

7 files changed

+11
-10
lines changed

packages/ats/sdk/__tests__/fixtures/beneficiary/BeneficiaryFixture.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export const AddBeneficiaryCommandFixture =
4545
createFixture<AddBeneficiaryCommand>((command) => {
4646
command.securityId.as(() => HederaIdPropsFixture.create().value);
4747
command.beneficiary.as(() => HederaIdPropsFixture.create().value);
48-
command.data.as(() => '0x');
48+
command.data?.as(() => '0x');
4949
});
5050

5151
export const RemoveBeneficiaryCommandFixture =

packages/ats/sdk/__tests__/fixtures/bond/BondFixture.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,6 @@ import {
258258
import AddBeneficiaryRequest from '@port/in/request/bond/AddBeneficiaryRequest';
259259
import RemoveBeneficiaryRequest from '@port/in/request/bond/RemoveBeneficiaryRequest';
260260
import UpdateBeneficiaryDataRequest from '@port/in/request/bond/UpdateBeneficiaryDataRequest';
261-
import GetBeneficiariesDataRequest from '@port/in/request/bond/GetBeneficiaryDataRequest';
262261

263262
export const SetCouponCommandFixture = createFixture<SetCouponCommand>(
264263
(command) => {
@@ -783,7 +782,7 @@ export const AddBeneficiaryRequestFixture =
783782
createFixture<AddBeneficiaryRequest>((request) => {
784783
request.securityId.as(() => HederaIdPropsFixture.create().value);
785784
request.beneficiaryId.as(() => HederaIdPropsFixture.create().value);
786-
request.data.as(() => '0x');
785+
request.data?.as(() => '0x');
787786
});
788787
export const UpdateBeneficiaryDataRequestFixture =
789788
createFixture<UpdateBeneficiaryDataRequest>((request) => {

packages/ats/sdk/src/app/usecase/command/security/beneficiaries/addBeneficiary/AddBeneficiaryCommand.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ export class AddBeneficiaryCommand extends Command<AddBeneficiaryCommandResponse
184184
constructor(
185185
public readonly securityId: string,
186186
public readonly beneficiary: string,
187-
public readonly data: string,
187+
public readonly data?: string,
188188
) {
189189
super();
190190
}

packages/ats/sdk/src/app/usecase/command/security/beneficiaries/addBeneficiary/AddBeneficiaryCommandHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ export class AddBeneficiaryCommandHandler
212212
await this.contractService.getContractEvmAddress(securityId);
213213

214214
const beneficiaryEvmAddress: EvmAddress =
215-
await this.contractService.getContractEvmAddress(beneficiary);
215+
await this.accountService.getAccountEvmAddress(beneficiary);
216216

217217
await this.validationService.checkPause(securityId);
218218

packages/ats/sdk/src/port/in/request/bond/AddBeneficiaryRequest.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,11 +205,13 @@
205205

206206
import ValidatedRequest from '@core/validation/ValidatedArgs';
207207
import FormatValidation from '../FormatValidation';
208+
import { OptionalField } from '@core/decorator/OptionalDecorator';
208209

209210
export default class AddBeneficiaryRequest extends ValidatedRequest<AddBeneficiaryRequest> {
210211
securityId: string;
211212
beneficiaryId: string;
212-
data: string;
213+
@OptionalField()
214+
data?: string;
213215

214216
constructor({
215217
securityId,
@@ -218,7 +220,7 @@ export default class AddBeneficiaryRequest extends ValidatedRequest<AddBeneficia
218220
}: {
219221
securityId: string;
220222
beneficiaryId: string;
221-
data: string;
223+
data?: string;
222224
}) {
223225
super({
224226
beneficiaryId: FormatValidation.checkHederaIdFormatOrEvmAddress(),

packages/ats/sdk/src/port/out/TransactionAdapter.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1925,7 +1925,7 @@ export default abstract class TransactionAdapter
19251925
abstract addBeneficiary(
19261926
security: EvmAddress,
19271927
beneficiary: EvmAddress,
1928-
data: string,
1928+
data?: string,
19291929
securityId?: ContractId | string,
19301930
): Promise<TransactionResponse>;
19311931
abstract removeBeneficiary(

packages/ats/sdk/src/port/out/rpc/RPCTransactionAdapter.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3374,7 +3374,7 @@ export class RPCTransactionAdapter extends TransactionAdapter {
33743374
addBeneficiary(
33753375
security: EvmAddress,
33763376
beneficiary: EvmAddress,
3377-
data: string,
3377+
data?: string,
33783378
securityId?: ContractId | string,
33793379
): Promise<TransactionResponse> {
33803380
LogService.logTrace(
@@ -3386,7 +3386,7 @@ export class RPCTransactionAdapter extends TransactionAdapter {
33863386
this.getSignerOrProvider(),
33873387
),
33883388
'addBeneficiary',
3389-
[beneficiary.toString(), data],
3389+
[beneficiary.toString(), data ?? '0x'],
33903390
GAS.ADD_BENEFICIARY,
33913391
);
33923392
}

0 commit comments

Comments
 (0)