Skip to content

Add phonebook contacts-by-group read - #13

Merged
MarcLeinenDE merged 4 commits into
mainfrom
dev/sdk-phonebook-group-read
Sep 8, 2026
Merged

Add phonebook contacts-by-group read#13
MarcLeinenDE merged 4 commits into
mainfrom
dev/sdk-phonebook-group-read

Conversation

@MarcLeinenDE

@MarcLeinenDE MarcLeinenDE commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Adds client.phonebook.contacts_by_group() using the newly live-confirmed phonebook/getcontactbygroup request shape.

Scope:

  • public helper takes integer group, page_capacity, page_index
  • exact wire serialization uses strings for all three nested values
  • response reuses the existing PhonebookContactsResponse
  • validates types/ranges before network access
  • no phonebook write in this helper

Upstream:

Physical validation:

  • preliminary direct-call probe passed on ACIY.3 with integer result, integer contactcount, list contactlist
  • public high-level helper test test_phonebook_group_read passed 1/1 in 0.46 s
  • selected test group returned zero items

Privacy:

  • integration test selects a real group index but never prints group names, contact names, phone numbers or contact-list contents

Documentation:

  • focused docs/phonebook-group-read.md evidence/usage note added.

@MarcLeinenDE MarcLeinenDE changed the title WIP: add phonebook contacts-by-group read Add phonebook contacts-by-group read Sep 8, 2026
@MarcLeinenDE
MarcLeinenDE marked this pull request as ready for review September 8, 2026 13:18
@MarcLeinenDE
MarcLeinenDE merged commit 1a061b9 into main Sep 8, 2026
5 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.

1 participant