-
Notifications
You must be signed in to change notification settings - Fork 499
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
[Provider] Recraft AI #835
Conversation
@VisargD according to the documentation:
I have included a commented block of code. Should I include that or will the included be fine? |
We should let the provider itself take care of defaults and validation error. The commented code is not required. Gateway currently does not throw validation error for provider specific errors because maintaining parity would become a hassle. |
Removed commented code. |
Please move the PR to ready state for review once its done. |
Hey @Ajay-Satish-01 , following parameters are not added in the mapping:
So we can add these 3 mapping in the
Everything else LGTM! |
@VisargD I have added those parameters as well. Anything else that I should fix or add? |
Title:
Description: (optional)
Related Issues: (optional)