-
Notifications
You must be signed in to change notification settings - Fork 231
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
[Enhancement]: Improve Prompt Management Documentation #2247
[Enhancement]: Improve Prompt Management Documentation #2247
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ariantManager and DeploymentManager method(s)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @aybruhm ! I've added minor comment. I think the main thing is that the PR is not clean
…sistency across manager methods
OSS is inaccessible, thus the reason why the workflow is failing. |
… into cleanup/prompt-management-doc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @aybruhm !
0297b0b
into
mmabrouk/docs/AGE-1151-prompt-management-documentation
Description
This PR updates and improves the prompt management documentation to reflect recent changes in the Prompt SDK and backend. It includes updated examples, and additional guidance for using synchronous and asynchronous methods.
Related Issue
Closes AGE-1284
Additional Improvement
Prompt
model in SDK typesConfigManager
for consistency across manager methodtest_config_manager
SDK tests to make use of updated method naming