Skip to content

Conversation

@sbrossie
Copy link
Member

@sbrossie sbrossie commented Aug 4, 2025

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for the new Aviate Coupons feature by adding the API documentation and integrating it into the existing Kill Bill documentation site.

  • Adds comprehensive API documentation for the Aviate Coupons feature with endpoints for creating, listing, retrieving, and archiving coupons
  • Creates the new documentation file structure for aviate-coupons
  • Updates all existing documentation files to include aviate-coupons in their navigation menus

Reviewed Changes

Copilot reviewed 29 out of 30 changed files in this pull request and generated 3 comments.

File Description
source/includes/_aviate-coupons.md Complete API documentation for Aviate Coupons with endpoints, models, and examples
source/aviate-coupons.html.md Main documentation page configuration for the Aviate Coupons feature
Multiple .html.md files Navigation menu updates to include aviate-coupons across all documentation pages

|-----------------|----------------|----------|--------------|--------------------------|
| includeArchived | boolean | false | false | Include archived coupons in the response
| includeExpired | boolean | false | false | Include expired coupons in the response
---
Copy link

Copilot AI Aug 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The horizontal rule separator (---) appears in the middle of the query parameters table documentation, which breaks the table formatting. This should be removed or relocated.

Suggested change
---

Copilot uses AI. Check for mistakes.
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 4, 2025

Deploying apidocs-killbill-io with  Cloudflare Pages  Cloudflare Pages

Latest commit: a97efac
Status: ✅  Deploy successful!
Preview URL: https://5173eb44.apidocs-killbill-io.pages.dev
Branch Preview URL: https://aviate-coupons.apidocs-killbill-io.pages.dev

View logs

| `planList` | `array` | Applicable plan names |
| `createdDate` | `string` (date-time) | Date when the coupon was created |
| `archived` | `boolean` | Whether the coupon is archived |

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a particular reason why we have CouponInput from Coupon? In all the other APIs, we do not have separate resource objects for the input hence asking.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have the same thing for the Wallet, but I agree we may not be consistent across all our resources.

sbrossie and others added 4 commits August 5, 2025 10:15
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sbrossie sbrossie merged commit dd1e280 into master Aug 5, 2025
2 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 5, 2025
github-actions bot pushed a commit to reshmabidikar/slate that referenced this pull request Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants