Skip to content
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

source-opsgenie contribution from wdlowry #53701

Conversation

wdlowry
Copy link

@wdlowry wdlowry commented Feb 14, 2025

What

This PR updates source Opsgenie (source-opsgenie).

The contributor provided the following description of the change:

Added two additional fields to the incident stream that are shown in the sample response in the documentation (https://docs.opsgenie.com/docs/incident-api):

  • impactDetectDate
  • impactEndDate

Reviewer checklist

  • Resolve any merge conflicts and validate file diffs (make sure the PR only includes changes intended by the contributor)
  • After reviewing the changes, run the bump-version Airbyte-CI command locally to update the version of the connector according to the versioning guidelines. Add breakingChanges to metadata if necessary.
  • Ensure connector docs are up to date with any changes
  • Run /format-fix to resolve any formatting errors
  • Click into the CI workflows that wait for a maintainer to run them, which should trigger CI runs

Copy link

vercel bot commented Feb 14, 2025

@wdlowry is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@wdlowry
Copy link
Author

wdlowry commented Feb 14, 2025

Apparently the UI made some significant changes to the YAML files. I'm going to close this and will probably generate a PR directly from the fork instead of through the UI.

@wdlowry wdlowry closed this Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants