Skip to content

fix wrong field path in the openmodel webhook #107

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.

Sign up for GitHub

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

Jump to bottom
Merged
InftyAI-Agent merged 1 commit into from
Aug 27, 2024
Merged

fix wrong field path in the openmodel webhook #107

InftyAI-Agent merged 1 commit into from
Aug 27, 2024

Conversation

@carlory
Copy link
Member

@carlory carlory commented Aug 27, 2024

What this PR does / why we need it

dataSource has been renamed to source

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

Does this PR introduce a user-facing change?

Fix the wrong field path in the open model webhook: renamed `dataSource` to `source`

@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Aug 27, 2024
@carlory
Copy link
Member Author

carlory commented Aug 27, 2024

/kind bug
/cc @kerthcet

@InftyAI-Agent InftyAI-Agent added bug Categorizes issue or PR as related to a bug. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Aug 27, 2024
@InftyAI-Agent InftyAI-Agent requested a review from kerthcet August 27, 2024 11:09
@kerthcet
Copy link
Member

kerthcet commented Aug 27, 2024

/lgtm
/approve

Thanks @carlory

@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Aug 27, 2024
@InftyAI-Agent InftyAI-Agent merged commit fec135b into InftyAI:main Aug 27, 2024
@carlory carlory deleted the webhook branch August 27, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kerthcet kerthcet Awaiting requested review from kerthcet

Assignees

@kerthcet kerthcet

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. bug Categorizes issue or PR as related to a bug. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@carlory @kerthcet @InftyAI-Agent