Skip to content

feat: expose sub_account_name on Number resource#371

Merged
avinashp-plivo merged 4 commits into
masterfrom
FNT-370
May 26, 2026
Merged

feat: expose sub_account_name on Number resource#371
avinashp-plivo merged 4 commits into
masterfrom
FNT-370

Conversation

@avinashp-plivo

Copy link
Copy Markdown
Contributor

Summary

Adds subAccountName property to the Number resource model, mirroring the new sub_account_name JSON field added to GET /v1/Account/{auth_id}/Number/ listing responses.

Uses the same isset()-with-null-fallback pattern as the other optional fields in this constructor, so older API versions that don't return the field continue to deserialize cleanly.

Companion to:

Test plan

  • CI build green
  • Verified manually against a staging response that returns sub_account_name for subaccount-owned numbers and null for parent-account numbers (verified upstream on number_service)

Supersedes #370 (same change, opened from a personal fork by mistake).

avinashp-plivo and others added 3 commits May 18, 2026 20:02
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@avinashp-plivo avinashp-plivo merged commit 7cf5eff into master May 26, 2026
3 of 10 checks passed
@avinashp-plivo avinashp-plivo deleted the FNT-370 branch May 26, 2026 06:02
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