fix: repair/remove list_* methods on Apollo's removed nested routes - #20
Conversation
Apollo removed several nested sub-resource GET routes (now 404). Audit fixed:
- list_contact_tasks: /contacts/{id}/tasks (gone) -> search_tasks(contact_ids=[id]).
- list_account_jobs: /accounts/{id}/job_postings (gone) -> resolve the account's
organization_id, then /organizations/{org_id}/job_postings.
- list_contact_calls: REMOVED. /contacts/{id}/calls is gone and phone_calls/search
ignores contact_ids/person_ids, so there's no working way to list a contact's
calls via the API.
- list_account_news: REMOVED. /accounts/{id}/news is gone with no replacement.
Live-verified: list_contact_tasks and list_account_jobs return correct data.
BREAKING: removes ApolloClient.list_contact_calls and list_account_news.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
[Question] Potential pagination behavior change in the rewired “list_*” methods
|
|
[Improvement] Version bump note vs actual change PR description says “No version bump”, but this PR updates |
Context
A live audit of every endpoint (prompted by the deprecated people search) found Apollo has removed several nested sub-resource GET routes — they now 404.
Changes
list_contact_tasks/contacts/{id}/taskssearch_tasks(contact_ids=[id])(verified: filters 33→1)list_account_jobs/accounts/{id}/job_postingsorganization_id, reads/organizations/{org_id}/job_postings(verified: 395 postings)list_contact_calls/contacts/{id}/callsphone_calls/searchignorescontact_ids/person_ids)list_account_news/accounts/{id}/newsRemoves
ApolloClient.list_contact_callsandlist_account_news. A companion apollo-cli PR removes thecallsandnewscommands that used them.Testing
No version bump (release PR will bump — this is a breaking change → minor/major your call at release).
🤖 Generated with Claude Code