docs(event_handler): add bedrock agent resolver documentation#3602
Conversation
b498cc0 to
c729d29
Compare
|
There was a problem hiding this comment.
This new Resolver for Bedrock Agents is truly a game changer @rubenfonseca!
I left some comments and I think we need to make it more clear to the customers:
1 - Create a diagram explaining how the Bedrock Agent works with Lambda.
2 - Add the video you are creating to make clear the amazing value-added of this utility.
3 - We need to rethink some sections of this documentation. I think we are investing a lot of documentation in SwaggerUI and not enough in Bedrock Agent, and the problems that this new Resolver solves.
Thanks brother ❤️
|
Please retry analysis of this Pull-Request directly on SonarCloud |
Bedrock reviewI love the user-experience. It makes authoring Agents as easy as writing an API - straight to the point, readable by anyone. For documentation, these are key areas we discussed during docs review that can take the current documentation to the next level: Tip Mental model
Items
PS: @hjgraca please chime in if I missed anything from today's session. |
sthulb
left a comment
There was a problem hiding this comment.
There's a few places where you're using a singular agent as "Agent" (cap 'A') where it should be lower case. "Agent" refers to the feature not the concept (if that makes sense)
leandrodamascena
left a comment
There was a problem hiding this comment.
Hello @rubenfonseca! We have some wrong characters that are breaking the links. I made suggestions to correct it.
leandrodamascena
left a comment
There was a problem hiding this comment.
Hello @rubenfonseca! Another batch of small suggestions! Thanks.
|
|
@leandrodamascena sorry for the mess, everything should be fixed now |
leandrodamascena
left a comment
There was a problem hiding this comment.
Congrats to everyone who contributed to this PR. We always make a difference in the lives of our customers when we provide very clear and concise documentation.
APPROVED!




Issue number: #3326
Summary
Changes
This PR adds documentation to the Bedrock Agent resolver feature.
Note: to allow sharing content between REST and Bedrock Agents resolvers, some content was moved to shared files and imported in both sides.
User experience
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Is this a breaking change?
RFC issue number:
Checklist:
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.