-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(doc): more info and arch diagram for landing page #822
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Continue to review full report in Codecov by Sentry.
|
There is a meta discussion here on whether we want to explain rollups to users? My opinion is that users of Jstz needn't know about rollups. So we could simplify the diagram to simply have |
499d1c5
to
d11c29e
Compare
Part of the value proposition of Jstz is that it is secured by Tezos and is not just floating out there somewhere. I tried to simplify the explanation of what's going on -- how does this look, @johnyob ? |
Hey Tim 👋 We discussed this PR in our weekly meeting today. We believe explicitly linking Jstz to Tezos on the homepage of the documentation isn’t necessary, as most users don’t need to be aware of the underlying Tezos-specific architecture. Additionally, the current architecture diagram doesn’t align with the styling of Jstz’s documentation. To move forward with this PR, we suggest:
|
Ok, I've moved the Tezos-specific stuff to an Architecture page and attempted to dark-mode the diagram. What do you think? |
d11c29e
to
a4ac839
Compare
Review landing page, add architecture page, and add arch diagram: