Skip to content

Conversation

@brozorec
Copy link
Collaborator

Added the whole suit of docs for the new smart account module.

@brozorec brozorec self-assigned this Oct 27, 2025
@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for openzeppelin-docs-v2 ready!

Name Link
🔨 Latest commit 4d25460
🔍 Latest deploy log https://app.netlify.com/projects/openzeppelin-docs-v2/deploys/6908ddf6e94d6a00074e9681
😎 Deploy Preview https://deploy-preview-64--openzeppelin-docs-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@stevedylandev
Copy link
Collaborator

@brozorec Feel free to merge whenever you're ready! You should have the permissions

@ozgunozerk
Copy link
Collaborator

For the Smart Accounts root page, I think a diagram representing the Authorization Flow will be quite helpful.

I recall confusing policies and context rules sometime ago.

I think the diagram can highlight the following:

  • relationship between the context rule(s) and the smart account contracts -> if possible to have many to many, and one to many, we can display all of them in this diagram
  • hierarchy between the context rules, policies and verifiers

So that, people will immediately have the mental model of how to build these for themselves.

@ozgunozerk
Copy link
Collaborator

ozgunozerk commented Oct 30, 2025

I realized after going into Context Rules page, we already have this:
image

What I propose is actually similar, (with the internal details are omitted, and the hierarchy is more pronounced, along with many to many and one to many etc. relationships are displayed).

I strongly believe such a diagram should be in the main page to help people establish the big picture for Smart Accounts.

@ozgunozerk
Copy link
Collaborator

Aside from the comments, superb work 🎉

@brozorec brozorec requested a review from ozgunozerk November 3, 2025 16:26
Copy link
Collaborator

@ozgunozerk ozgunozerk left a comment

Choose a reason for hiding this comment

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

one unresolved issue, rest is great!

@brozorec brozorec merged commit 8e93a6a into main Nov 4, 2025
6 checks passed
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.

4 participants