Skip to content

Aviate wallet: Document MVP apis for wallet#339

Merged
sbrossie merged 5 commits into
masterfrom
aviate-wallet
Apr 15, 2025
Merged

Aviate wallet: Document MVP apis for wallet#339
sbrossie merged 5 commits into
masterfrom
aviate-wallet

Conversation

@sbrossie
Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 15, 2025

Deploying apidocs-killbill-io with  Cloudflare Pages  Cloudflare Pages

Latest commit: 306a833
Status:⚡️  Build in progress...

View logs

Copy link
Copy Markdown
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.

Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (2)

source/includes/_aviate-wallet.md:3

  • Found a typo: 'acounts' should be 'accounts'.
The [Aviate Wallet feature](https://docs.killbill.io/latest/aviate-wallet) provides several endpoints that allow you to manage the wallets associated with user acounts.

source/admin.html.md:32

  • [nitpick] The added menu item 'aviate-wallet' contains trailing spaces. Removing extra spacing will improve consistency.
  - aviate-wallet  

Comment thread source/includes/_aviate-wallet.md Outdated
Comment thread source/includes/_aviate-wallet.md Outdated
sbrossie and others added 2 commits April 14, 2025 21:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread source/includes/_aviate-wallet.md Outdated
| `expDate` | `string` (ISO datetime) | user | Optional expiration timestamp |
| `parentRecordId` | `integer` | system | Reference to the original credit record (for partial consumption) |
| `kbInvoiceId` | `string` (UUID)| system | Associated Kill Bill invoice ID (if credit was paid) |
| `kbPaymentId` | `string` (UUID)| system | Associated Kill Bill payment ID |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(if credit was paid)?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in fc91c9a

@sbrossie sbrossie merged commit 8e3f445 into master Apr 15, 2025
1 of 2 checks passed
github-actions Bot pushed a commit that referenced this pull request Apr 15, 2025
github-actions Bot pushed a commit to reshmabidikar/slate that referenced this pull request Apr 22, 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