updated image src paths for moved files

[eric-stacks]

Description

in the previous PR merged, some files were moved to a new Space folder but the image src paths were not adjusted. this PR fixes that.

[claude]

PR Review: Image Path Updates

Summary

This PR successfully fixes broken image references that occurred when files were moved from docs/build/ to docs/learn/ in a previous PR. The changes update image source paths and reorganize assets to match the new directory structure.

What Changed

• 34 image files moved from docs/build/.gitbook/assets/ to docs/learn/.gitbook/assets/
• Images renamed with descriptive suffixes for better organization:
  • -with-fordefi.png suffix for Fordefi wallet tutorial images
  • -with-asigna.png suffix for Asigna wallet tutorial images
  • -sbtc-bridge.png suffix for general sBTC bridge images
• 3 markdown files updated with corrected image paths:
  • how-to-use-the-sbtc-bridge-with-asigna.md (15 path updates)
  • how-to-use-the-sbtc-bridge-with-fordefi.md (15 path updates)
  • how-to-use-the-sbtc-bridge.md (6 path updates)

Code Quality Assessment

✅ Strengths:

1. Correct path updates: All relative paths properly changed from ../ to ../../ to reflect the new directory structure
2. Consistent naming convention: Images now have descriptive suffixes that clearly indicate which tutorial they belong to
3. Complete coverage: All image references in the affected markdown files have been updated
4. No functional changes: This is purely a maintenance PR fixing broken links

✅ Best Practices Followed:

• Atomic change focused on a single issue (broken image paths)
• Clear PR description explaining the problem and solution
• Organized file structure with assets properly grouped under the correct documentation section

Verification

I verified that:

• All image path changes follow the correct relative path structure (../../.gitbook/assets/ from the markdown file locations)
• The markdown files are located at docs/learn/sbtc/walkthroughs/
• The assets are correctly placed at docs/learn/.gitbook/assets/
• The path structure ../../.gitbook/assets/ correctly navigates up two levels (from walkthroughs/ to sbtc/ to learn/) and then into .gitbook/assets/

Security Considerations

✅ No security concerns - this PR only moves image files and updates references

Performance Considerations

✅ No performance impact - file moves and path updates do not affect runtime performance

Test Coverage

N/A - This is a documentation-only change. Manual verification by viewing the documentation pages would be appropriate to ensure images display correctly.

Recommendation

Approve and merge - This PR correctly fixes the broken image references introduced in the previous file reorganization. The changes are straightforward, complete, and follow good naming conventions.

Suggested Follow-up Actions (Optional)

After merging, consider:

1. Manually verifying the affected documentation pages render correctly with all images displaying
2. Checking if any other documentation sections were affected by the previous file move that may need similar fixes

[claude]

PR Review: Updated Image Source Paths

Summary

This PR successfully fixes broken image links in sBTC Bridge documentation by updating image source paths to reflect the new directory structure after files were moved from docs/build/.gitbook/assets/ to docs/learn/.gitbook/assets/.

Code Quality ✅

Strengths:

• Systematic approach: All 34 image files were moved consistently and renamed with descriptive suffixes (-with-fordefi, -with-asigna, -sbtc-bridge)
• Improved naming convention: The new naming scheme with contextual suffixes makes images more identifiable and reduces naming conflicts
• Complete coverage: All references in the three affected markdown files were updated to match the new paths
• Path consistency: All image references now use relative paths correctly (../../.gitbook/assets/ instead of ../.gitbook/assets/)

Changes breakdown:

• 34 binary image files moved from docs/build/ to docs/learn/ directory
• 3 markdown files updated with corrected image paths:
  • how-to-use-the-sbtc-bridge-with-asigna.md (15 references)
  • how-to-use-the-sbtc-bridge-with-fordefi.md (15 references)
  • how-to-use-the-sbtc-bridge.md (6 references)

Potential Issues 🔍

None identified - This is a straightforward file move and path update with no code logic changes.

Best Practices ✅

1. Clear PR description - Explains the context (previous PR moved files but didn't update paths)
2. Consistent renaming - Adding context to filename helps with asset management
3. Relative path usage - Properly uses relative paths from the document location

Testing Recommendations 🧪

Before merging, verify:

1. All images render correctly in the documentation viewer/GitBook
2. No broken image links appear in any of the three walkthrough pages
3. The relative paths work correctly from the docs/learn/sbtc/walkthroughs/ location

Performance Considerations ⚡

• No performance impact - this is purely a file organization change
• Binary files are unchanged, only moved/renamed

Security Concerns 🔒

• No security implications - documentation-only changes with no code execution

Documentation Impact 📚

Positive impact: This PR ensures the sBTC Bridge documentation displays correctly with all tutorial images properly rendered, which is critical for user guidance through the multi-signature wallet setup processes.

---

Recommendation: ✅ Approve and merge

This is a clean fix that resolves broken image links without introducing any risks. The improved naming convention is a bonus that makes asset management easier going forward.

[kenny-stacks]

lgtm