-
Notifications
You must be signed in to change notification settings - Fork 459
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
Update II mobile integration sequence diagrams to use mermaid #3930
base: master
Are you sure you want to change the base?
Conversation
Update II mobile integration sequence diagrams to use mermaid
🤖 Here's your preview: https://un2ts-zaaaa-aaaam-abxuq-cai.icp0.io |
If anyone (@robin-kunzler @venkkatesh-sekar) has a nicer way to add a caption or title to the sequence diagram, please go ahead. The notes on the sequence diagrams can only be changed color globally so unless we really oppose the fact they are not as visible anymore I'd avoid doing that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Approved modulo a single comment.
PWA ->> II_FE: 4. Standard II client auth protocol | ||
U ->> II_FE: 5. Authenticate with passkey | ||
activate II_BE | ||
II_FE ->> II_BE: 6. getDelegation(frontEndHostname) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not clear here that the delegation is granted for the session public key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a note on line 145 similar to the second diagram.
Update II mobile integration sequence diagrams to use mermaid
Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:
.mdx
file format to support the previous two components./sidebars.js
, otherwise, it will not appear in theside navigation bar.
.github/CODEOWNERS
file isfilled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.