-
Notifications
You must be signed in to change notification settings - Fork 188
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
docs(aws-bedrock): AWS bedrock documentation #4032
Conversation
…ws bedrock connector
👋 🤖 🤔 Hello! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.5/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
…drock.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…drock.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Hi @mathias-vandaele 😄 Is this linked to a particular issue, and set for a particular alpha or release? |
Hi @christinaausley This should be release on the next alpha, which should be 8.6.0-alpha4 |
@mathias-vandaele Thanks for the details! Just let me know when you're ready for a technical review and an engineering review. |
Hey @christinaausley I am ready for a technical and engineering review, as It should now be in the last release |
docs/components/connectors/out-of-the-box-connectors/amazon-bedrock.md
Outdated
Show resolved
Hide resolved
docs/components/connectors/out-of-the-box-connectors/amazon-bedrock.md
Outdated
Show resolved
Hide resolved
`newMessage` is the latest message written by the assistant | ||
|
||
Ideally, the messages history needs to transit within the process and needs to be the input of this `Converse` task. | ||
With the new message |
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.
I think this was either carried down a line or an incomplete sentence, but made an adjustment in my latest commit.
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.
This will need to be added to the sidebars.js file 👍
I have done it @christinaausley |
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.
I moved it around in the sidebar to accommodate the alphabetical style. Nice work!
@mathias-vandaele @sbuettner I assume this is ready for release given the review discussion -- merging. |
add complete documentation on how to use the new aws bedrock connector
Linked Issue: https://github.com/camunda/team-connectors/issues/842
Will be release on the next alpha, which should be 8.6.0-alpha4
Description
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).