docs(source-incident-io): improve documentation accuracy and add escalations stream#74073
Draft
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
Draft
docs(source-incident-io): improve documentation accuracy and add escalations stream#74073devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
Conversation
…lations stream Co-Authored-By: bot_apk <apk@cognition.ai>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
| @@ -34,19 +33,20 @@ The source connector supports the following [sync modes](https://docs.airbyte.co | |||
| | Stream Name | Primary Key | Pagination | Supports Full Sync | Supports Incremental | | |||
Contributor
There was a problem hiding this comment.
[markdownlint-fix] reported by reviewdog 🐶
Suggested change
| | Stream Name | Primary Key | Pagination | Supports Full Sync | Supports Incremental | | |
| | Stream Name | Primary Key | Pagination | Supports Full Sync | Supports Incremental | |
Contributor
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 2aef3c4. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Improves the Incident.io source connector documentation, triggered by #74072 which added the
escalationsstream and migratedincident_rolesto the v2 API.Requested by: bot_apk (apk@cognition.ai)
Link to Devin run: https://app.devin.ai/sessions/91cde89703274cae981b49ef04c9a366
How
Single file change:
docs/integrations/sources/incident-io.mdFixes to incorrect information:
extendsheading" — corrected to Settings > API keys with a link toapp.incident.io/settings/api-keys, verified against incident.io auth docsAdded missing information:
escalationsstream (new in v0.1.21, uses DefaultPaginator) to the Streams tableImproved clarity:
Review guide
docs/integrations/sources/incident-io.md— the only file changedKey things to verify:
escalationswas added; confirm no streams were accidentally dropped during the alphabetical re-sortUser Impact
Users see more accurate setup instructions and a complete list of available streams including the new
escalationsstream.Can this PR be safely reverted and rolled back?