-
Notifications
You must be signed in to change notification settings - Fork 349
Expose ResponseContentPart.InputImageUrl
#874
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi @crickman. Thanks very much for your contribution and your interest in improving the OpenAI developer experience. As discussed in #875, I think the proposed change aligns with the local and upstream REST API specs and agree this should be public. We appreciate your thorough and comprehensive approach to test validation. The implementation looks good to me. I'm going to ask @joseharriaga to make the final call, as he'll have a more complete understanding of the historical context. |
|
Hi @crickman - Please run the Export-API script to update the public API surface file. |
I'm getting this error: Is there a module I need to install (type-spec related maybe)? |
I think you can grab it from here https://github.com/dotnet/sdk/tree/main/src/Compatibility/GenAPI |
|
@christothes - Updated |
Expose
InternalItemContentInputImage.ImageUrlas public property:ResponseContentPart.InputImageUrlAdded smoke validation all public
ResponseContentPartusage and serializationcloses #875