chore: remove service token endpoint which is unused - #8797
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe changes remove service-level API token functionality by deleting the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR removes an unused “service API token” endpoint from the API surface, along with its view export and URL route, reducing dead code in the API layer.
Changes:
- Deleted
ServiceApiTokenEndpointfromapps/api/plane/app/views/api.pyand removed now-unneeded imports. - Removed the corresponding view export from
apps/api/plane/app/views/__init__.py. - Removed the
"workspaces/<str:slug>/service-api-tokens/"URL pattern fromapps/api/plane/app/urls/api.py.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| apps/api/plane/app/views/api.py | Removes the unused service-token endpoint implementation and related imports. |
| apps/api/plane/app/views/init.py | Stops exporting the removed endpoint from the views package. |
| apps/api/plane/app/urls/api.py | Removes the URL route and import for the removed endpoint. |
Description
Type of Change
- [ ] Code refactoring
Summary by CodeRabbit