Skip to content

Profile: embed wallet layers diagram, update component table - #35

Merged
leifj merged 1 commit into
mainfrom
diagrams-redraw
Oct 20, 2026
Merged

leifj merged 1 commit into
mainfrom
diagrams-redraw

Conversation

@leifj

@leifj leifj commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Embeds the wallet layers diagram below the platform architecture diagram.
  • Splits the component table: Web Wallet (wallet-frontend, wallet-companion), Wrapper Apps (wallet-android-wrapper, wallet-ios-wrapper, added) and Native SDKs, described consistently with the diagrams.

Ordering

Do not merge before sirosfoundation/docs#84 is merged and deployed. Both images are loaded from https://developers.siros.org/img/; wallet-layers.svg does not exist at that URL until the docs PR is live, and the overview there is still the old drawing.

🤖 Generated with Claude Code

…ps and SDKs

Depends on sirosfoundation/docs#84 being merged and deployed, which provides
the images at developers.siros.org/img/.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The required docs PR #84 remains an unmerged draft and must be merged and deployed first.

No open issues 🎉

What changed in this PR

Updates the organization profile to clarify wallet architecture and component roles.

Changes:

  • Embeds the wallet layers diagram below the platform architecture.
  • Separates Web Wallet, Wrapper Apps, and Native SDKs in the component table.
File Description
profile/​README.md Adds the wallet diagram and updates component descriptions and repository links.

Review effort: Balanced

@leifj
leifj marked this pull request as ready for review October 20, 2026 21:49
@leifj
leifj merged commit 448f99a into main Oct 20, 2026
1 check passed
@leifj
leifj deleted the diagrams-redraw branch October 20, 2026 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants