Skip to content

fix: use correct operation name for invoice v3 get by id#4584

Merged
tothandras merged 1 commit into
mainfrom
fix/invoice-v3-op-name
Jun 26, 2026
Merged

fix: use correct operation name for invoice v3 get by id#4584
tothandras merged 1 commit into
mainfrom
fix/invoice-v3-op-name

Conversation

@borosr

@borosr borosr commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Overview

Fixes #(issue)

Notes for reviewer

Summary by CodeRabbit

  • Bug Fixes
    • Improved invoice retrieval handling for billing requests.

@borosr
borosr requested a review from a team as a code owner June 26, 2026 12:24
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3ed0ef56-3a76-43ce-bdb6-d15a637071ed

📥 Commits

Reviewing files that changed from the base of the PR and between c725b4d and 8ce7e08.

📒 Files selected for processing (1)
  • api/v3/handlers/billinginvoices/get.go

📝 Walkthrough

Walkthrough

The billing invoice GET handler now registers the HTTP transport operation name as get-invoice instead of get-billing-invoice.

Changes

Billing invoice handler metadata

Layer / File(s) Summary
HTTP transport operation name update
api/v3/handlers/billinginvoices/get.go
The handler’s httptransport.WithOperationName(...) value changes from get-billing-invoice to get-invoice.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 matches the change: it describes correcting the v3 invoice get-by-id operation name.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/invoice-v3-op-name

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.

❤️ Share

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

@borosr
borosr requested a review from tothandras June 26, 2026 12:25
@borosr borosr self-assigned this Jun 26, 2026
@borosr borosr added the release-note/bug-fix Release note: Bug Fixes label Jun 26, 2026
@borosr
borosr enabled auto-merge (squash) June 26, 2026 12:25
@greptile-apps

greptile-apps Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR corrects the runtime operation name for the v3 invoice get-by-id handler. The main change is:

  • Updates GetBillingInvoice to use get-invoice as the transport operation name.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
api/v3/handlers/billinginvoices/get.go Changes the handler metadata from get-billing-invoice to the spec-aligned get-invoice operation name.

Reviews (1): Last reviewed commit: "fix: use correct operation name for invo..." | Re-trigger Greptile

@tothandras
tothandras disabled auto-merge June 26, 2026 12:26
@tothandras
tothandras merged commit a693618 into main Jun 26, 2026
26 of 27 checks passed
@tothandras
tothandras deleted the fix/invoice-v3-op-name branch June 26, 2026 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/bug-fix Release note: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants