Skip to content

fix: /role endpoint to /roles - #115

Merged
nitisht merged 1 commit into
parseablehq:mainfrom
pratik50:fix/roleEndpoint
Aug 17, 2026
Merged

fix: /role endpoint to /roles#115
nitisht merged 1 commit into
parseablehq:mainfrom
pratik50:fix/roleEndpoint

Conversation

@pratik50

@pratik50 pratik50 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Changes

  • /role endpoint is deprecated, replacing with /roles

Summary by CodeRabbit

  • Bug Fixes
    • Corrected role retrieval to use the proper roles endpoint, improving successful loading of available roles.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ed683570-40e4-48e9-916b-cc421a303f9b

📥 Commits

Reviewing files that changed from the base of the PR and between 25b275f and ec65f30.

📒 Files selected for processing (1)
  • cmd/role.go

Included review availability: 4 reviews are currently available. Based on recent review activity, included reviews refill at 5 per hour.


📝 Walkthrough

Walkthrough

fetchRoles now requests the role collection from the roles endpoint. Response parsing and error handling remain unchanged.

Changes

Role retrieval

Layer / File(s) Summary
Use the roles collection endpoint
cmd/role.go
fetchRoles changes its GET request path from role to roles. Response parsing and error handling remain unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to ec65f

This is a localized endpoint rename from /role to /roles; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: nitisht

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the endpoint change from /role to /roles, which matches the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@pratik50 pratik50 self-assigned this Aug 17, 2026
@nitisht
nitisht merged commit 25493c9 into parseablehq:main Aug 17, 2026
3 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants