Skip to content

Commit

Permalink
Merge pull request #296 from digitalcz/DGS-3010-multisign-to-bulksign…
Browse files Browse the repository at this point in the history
…ature-renaming

Renamed MultiSing into BulkSignature
  • Loading branch information
IamDejv authored Feb 4, 2025
2 parents 321b750 + 0279afe commit c60cd43
Show file tree
Hide file tree
Showing 11 changed files with 46 additions and 46 deletions.
6 changes: 3 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ Updates should follow the [Keep a CHANGELOG](http://keepachangelog.com/) princip
- Edit `AccountGuide.settingsFilled`
- Add `MyPreferences.guideHide`
- Add `Account.guide` endpoint and `AccountGuide` resource
- Add `AccountBilling.multiSign`
- Edit `MultiSignRecipientInfo` resource
- Add `AccountBilling.bulkSigning`
- Edit `BulkSignatureRecipientInfo` resource
- Add `BatchSendingsEndpoint.MutiSignRecipients` endpoint
- Add `MyEndpoint.updatePreferences` endpoint and `MyPreferences` resource
- Add `SignatureScenarioVersionInfo.role`
Expand All @@ -28,7 +28,7 @@ Updates should follow the [Keep a CHANGELOG](http://keepachangelog.com/) princip
- Add `AccountEndpoint.createDefaultSubscription` endpoint
- Add `SignatureScenario.latestVersionInfo`
- Add `EnvelopeTemplateTag`.`customIdentifier` in to resource
- Add `MultiSign.get` endpoint
- Add `BulkSignature.get` endpoint

## [2.5.0] - 2024-10-14
- Add `AccountBilling.automaticTagsPlacement`
Expand Down
6 changes: 3 additions & 3 deletions src/DigiSign.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
use DigitalCz\DigiSign\Endpoint\AccountEndpoint;
use DigitalCz\DigiSign\Endpoint\AuthEndpoint;
use DigitalCz\DigiSign\Endpoint\BatchSendingsEndpoint;
use DigitalCz\DigiSign\Endpoint\BulkSignatureEndpoint;
use DigitalCz\DigiSign\Endpoint\DeliveriesEndpoint;
use DigitalCz\DigiSign\Endpoint\EndpointInterface;
use DigitalCz\DigiSign\Endpoint\EnumsEndpoint;
Expand All @@ -19,7 +20,6 @@
use DigitalCz\DigiSign\Endpoint\IdentificationsEndpoint;
use DigitalCz\DigiSign\Endpoint\ImagesEndpoint;
use DigitalCz\DigiSign\Endpoint\LabelsEndpoint;
use DigitalCz\DigiSign\Endpoint\MultiSignEndpoint;
use DigitalCz\DigiSign\Endpoint\MyEndpoint;
use DigitalCz\DigiSign\Endpoint\ReportEndpoint;
use DigitalCz\DigiSign\Endpoint\WebhooksEndpoint;
Expand Down Expand Up @@ -290,9 +290,9 @@ public function report(): ReportEndpoint
return new ReportEndpoint($this);
}

public function multiSign(): MultiSignEndpoint
public function bulkSignature(): BulkSignatureEndpoint
{
return new MultiSignEndpoint($this);
return new BulkSignatureEndpoint($this);
}

private function createUserAgent(): string
Expand Down
10 changes: 5 additions & 5 deletions src/Endpoint/BatchSendingsEndpoint.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@
use DigitalCz\DigiSign\Resource\BaseResource;
use DigitalCz\DigiSign\Resource\BatchSending;
use DigitalCz\DigiSign\Resource\BatchSendingStats;
use DigitalCz\DigiSign\Resource\BulkSignatureRecipientInfo;
use DigitalCz\DigiSign\Resource\Collection;
use DigitalCz\DigiSign\Resource\ListResource;
use DigitalCz\DigiSign\Resource\MultiSignRecipientInfo;

/**
* @extends ResourceEndpoint<BatchSending>
Expand Down Expand Up @@ -55,13 +55,13 @@ public function stats(BatchSending|string $id): BatchSendingStats
}

/**
* @return Collection<MultiSignRecipientInfo>
* @return Collection<BulkSignatureRecipientInfo>
*/
public function multiSignRecipients(BatchSending|string $id): Collection
public function bulkSignatureRecipients(BatchSending|string $id): Collection
{
return $this->createCollectionResource(
$this->getRequest('/{id}/multi-sign-recipients', ['id' => $id]),
MultiSignRecipientInfo::class,
$this->getRequest('/{id}/bulk-signature-recipients', ['id' => $id]),
BulkSignatureRecipientInfo::class,
);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,17 @@

use DigitalCz\DigiSign\DigiSign;
use DigitalCz\DigiSign\Endpoint\Traits\GetEndpointTrait;
use DigitalCz\DigiSign\Resource\MultiSign;
use DigitalCz\DigiSign\Resource\BulkSignature;

/**
* @extends ResourceEndpoint<MultiSign>
* @extends ResourceEndpoint<BulkSignature>
*/
final class MultiSignEndpoint extends ResourceEndpoint
final class BulkSignatureEndpoint extends ResourceEndpoint
{
use GetEndpointTrait;

public function __construct(DigiSign $parent)
{
parent::__construct($parent, '/api/multi-signs', MultiSign::class);
parent::__construct($parent, '/api/bulk-signatures', BulkSignature::class);
}
}
2 changes: 1 addition & 1 deletion src/Resource/AccountBilling.php
Original file line number Diff line number Diff line change
Expand Up @@ -66,5 +66,5 @@ class AccountBilling extends BaseResource

public bool $automaticTagsPlacement;

public bool $multiSign;
public bool $bulkSigning;
}
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
use DateTime;
use DigitalCz\DigiSign\Resource\Traits\EntityResourceTrait;

final class MultiSign extends BaseResource
final class BulkSignature extends BaseResource
{
use EntityResourceTrait;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

namespace DigitalCz\DigiSign\Resource;

class MultiSignRecipientInfo extends BaseResource
class BulkSignatureRecipientInfo extends BaseResource
{
public string $name;

Expand All @@ -14,5 +14,5 @@ class MultiSignRecipientInfo extends BaseResource

public int $count;

public bool $isMultiSignable;
public bool $isBulkSignable;
}
4 changes: 2 additions & 2 deletions tests/DigiSignTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -156,8 +156,8 @@ public function testChildren(): void
self::assertSame('/api/my', $mockClient->getLastRequest()->getUri()->getPath());
$dgs->report()->request('GET');
self::assertSame('/api/report', $mockClient->getLastRequest()->getUri()->getPath());
$dgs->multiSign()->request('GET');
self::assertSame('/api/multi-signs', $mockClient->getLastRequest()->getUri()->getPath());
$dgs->bulkSignature()->request('GET');
self::assertSame('/api/bulk-signatures', $mockClient->getLastRequest()->getUri()->getPath());
}

public function testUserAgent(): void
Expand Down
6 changes: 3 additions & 3 deletions tests/Endpoint/BatchSendingEndpointTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@ public function testStats(): void
self::assertLastRequest('GET', "/api/batch-sendings/foo/stats");
}

public function testMultiSignRecipients(): void
public function testBulkSignatureRecipients(): void
{
self::endpoint()->multiSignRecipients('foo');
self::assertLastRequest('GET', "/api/batch-sendings/foo/multi-sign-recipients");
self::endpoint()->bulkSignatureRecipients('foo');
self::assertLastRequest('GET', "/api/batch-sendings/foo/bulk-signature-recipients");
}

protected static function endpoint(): BatchSendingsEndpoint
Expand Down
22 changes: 22 additions & 0 deletions tests/Endpoint/BulkSignatureEndpointTest.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
<?php

declare(strict_types=1);

namespace DigitalCz\DigiSign\Endpoint;

/**
* @covers \DigitalCz\DigiSign\Endpoint\BulkSignatureEndpoint
*/
final class BulkSignatureEndpointTest extends EndpointTestCase
{
public function testGet(): void
{
self::endpoint()->get('foo');
self::assertLastRequest('GET', '/api/bulk-signatures/foo');
}

protected static function endpoint(): BulkSignatureEndpoint
{
return self::dgs()->bulkSignature();
}
}
22 changes: 0 additions & 22 deletions tests/Endpoint/MultiSignEndpointTest.php

This file was deleted.

0 comments on commit c60cd43

Please sign in to comment.