Skip to content

TING-962 API explicit document ordering #103

Merged
yeu1 merged 1 commit into
masterfrom
TING-962
Jun 24, 2026
Merged

TING-962 API explicit document ordering #103
yeu1 merged 1 commit into
masterfrom
TING-962

Conversation

@yeu1

@yeu1 yeu1 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Add documentOrder for documents model

…ApS (5542) PHP SDK v4.0.0

Add documentOrder for documents model
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 8d13c02a-7c8f-4113-ae2a-253679241ee6

📥 Commits

Reviewing files that changed from the base of the PR and between 3d252c9 and 0316671.

📒 Files selected for processing (3)
  • CHANGELOG.md
  • docs/document.md
  • src/Document.php

Walkthrough

The Document class in src/Document.php gains a protected $documentOrder = 0 field, with the field added to the $propertyMapping array for both create and update operations. Two new public methods are introduced: getDocumentOrder(): int and setDocumentOrder(int $documentOrder): void. The documentation in docs/document.md is updated to include a setDocumentOrder(0) call in the creation example and new state variable entries describing both methods. The CHANGELOG.md records these additions under the Unreleased section.


Comment @coderabbitai help to get the list of available commands.

@sorinpuncea sorinpuncea left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@yeu1 yeu1 changed the title TING-962 API Document Ordering Broken for customer Northern Partners … TING-962 API explicit document ordering Jun 24, 2026
@yeu1 yeu1 merged commit dc861ae into master Jun 24, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants