fix(payments): remove duplicate T01 flow diagrams#1669
Closed
fahadfa-aws wants to merge 1 commit into
Closed
Conversation
T01 had four PNGs but only two distinct flows. The strands_payment_flow and strands_agent_payment_flow PNGs render the same sequence diagram — the only pixel difference is "AgentCore Payments" vs "AgentCore payments" on the right-most lane label. Same for the LangGraph pair. The README referenced both copies in each Architecture subsection, which made it look like two diagrams while showing the same content twice. - Delete strands_payment_flow.png and langgraph_payment_flow.png - Drop the orphan README references at lines 25 and 49 Keeps the *_agent_payment_flow.png variants because their lane label matches the lowercase "AgentCore payments" spelling used elsewhere in the tutorial set.
Author
|
@mvangara10 — flagging this for your review when you have a moment. Tagged across the full set of payments-tutorial fixes I've been pushing today; happy to walk through any of them. Audit logs and test evidence are referenced in the PR description. |
Author
|
Superseded by #1738 (consolidated PR) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue
Tutorial 01 (
01-agents-payments-and-limits/) ships four flow PNGs but only two distinct flows. Each pair renders the same sequence diagram twice — the only pixel difference betweenstrands_payment_flow.pngandstrands_agent_payment_flow.pngis the casing of "AgentCore Payments" vs "AgentCore payments" on the right-most lane label. Same for the LangGraph pair. The README referenced both copies in each Architecture subsection, which presented as two diagrams to the reader while showing the same content twice.Verified by reading both PNGs as images: arrows, lane labels, and message text are identical apart from the one-letter casing slip on the right-most lane.
Changes
images/strands_payment_flow.pngandimages/langgraph_payment_flow.png.Keeps the
*_agent_payment_flow.pngvariants because their lane label uses lowercase "AgentCore payments" — matches the spelling used in the Tutorial 00 README title and the AWS documentation.Docs/assets only. No code change.