-
Notifications
You must be signed in to change notification settings - Fork 16
feat: add support to expand webhooks and callbacks in toc #216
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 webhooks and callbacks in the table of contents (TOC) generation by introducing two new flags --expand-webhooks and --expand-callbacks to the portal toc new command.
- Adds new TOC types for webhooks and callbacks with group overview support
- Extracts webhook and callback data from SDL specification files
- Integrates webhook and callback support into the TOC structure generation process
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/utils.ts | Adds utility functions for title case conversion and unique group name generation |
| src/types/toc/toc.ts | Defines new webhook and callback TOC interfaces and types |
| src/types/sdl/sdl.ts | Extends SDL types and adds extraction functions for webhooks and callbacks |
| src/prompts/portal/toc/new-toc.ts | Updates prompts to handle multiple component types |
| src/commands/portal/toc/new.ts | Adds CLI flags for expanding webhooks and callbacks |
| src/application/portal/toc/toc-structure-generator.ts | Integrates webhook and callback support into TOC structure generation |
| src/actions/portal/toc/new-toc.ts | Updates action to handle new webhook and callback parameters |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…thub.com/apimatic/apimatic-cli into ayeshas09/add-webhooks-callbacks-support
…thub.com/apimatic/apimatic-cli into ayeshas09/add-webhooks-callbacks-support
…thub.com/apimatic/apimatic-cli into ayeshas09/add-webhooks-callbacks-support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…thub.com/apimatic/apimatic-cli into ayeshas09/add-webhooks-callbacks-support
…thub.com/apimatic/apimatic-cli into ayeshas09/add-webhooks-callbacks-support
|



This PR adds support for the following flags in the
portal toc newcommand:--expand-webhooks
--expand-callbacks