Rename unify.pm.issues to unify.ticketing.tickets#2
Conversation
Mirror the Unify API migration from the /pm/issues endpoint to /ticketing/tickets: - Rename bundleup/unify/pm.py -> ticketing.py (PM class -> Ticketing, issues() -> tickets(), path pm/issues -> ticketing/tickets) - Update the unify.ticketing accessor in __init__.py - Update tests, README, and examples Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014zVgT6VdGxc5a96oMxJU87
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 52f6d55889
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| """ | ||
| List ticketing tickets | ||
| """ | ||
| url = self._build_url("ticketing/tickets") |
There was a problem hiding this comment.
Point ticketing calls at the published PM endpoint
The currently published BundleUp Unified API reference still documents the Jira/Linear issue list route as GET /v1/pm/issues (https://bundleup.mintlify.app/api-reference/project-management/list-issues), not /v1/ticketing/tickets. With this path, every unify.ticketing.tickets() call against the documented production API will hit an undocumented route and fail with 404 instead of returning issues, unless the server-side migration is deployed first or a compatibility alias is added.
Useful? React with 👍 / 👎.
Mirrors the Unify API migration from the
/pm/issuesendpoint to/ticketing/tickets.Changes
bundleup/unify/pm.py→bundleup/unify/ticketing.py:PMclass →Ticketing,issues()→tickets()(pathpm/issues→ticketing/tickets)bundleup/unify/__init__.py:self.pm→self.ticketingtests/test_unify.py,tests/test_bundleup.py), README, and examplesVerification
pytest— all 87 tests passscripts/check-sdk-parity.jsin the API repo reports no missingticketing.ticketsmethod for this SDKNote
This SDK was already missing the
crmanddrivenamespaces and thechat.users/chat.message/git.branchesmethods relative to the canonical API surface. That drift is pre-existing and out of scope for this change.🤖 Generated with Claude Code
https://claude.ai/code/session_014zVgT6VdGxc5a96oMxJU87
Generated by Claude Code