Skip to content
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

Docs: Add high level info about transaction lifecycle #3764

Closed
Tracked by #2658
critesjosh opened this issue Dec 21, 2023 · 0 comments · Fixed by #3763
Closed
Tracked by #2658

Docs: Add high level info about transaction lifecycle #3764

critesjosh opened this issue Dec 21, 2023 · 0 comments · Fixed by #3763
Assignees
Labels
A-documentation Area: relates to documentation T-feedback Type: recording user feedback

Comments

@critesjosh
Copy link
Contributor

Feedback:
One section I believe could improve developers', and even users', understanding/intuition of the private inner workings of Aztec is one dedicated to explaining what happens, for example, when a user calls a private function in a very thorough yet ELI5 way. When I say thorough, I mean explaining every piece involved in the interaction, from the user device to the wallet to the trees, and so on - the complete flow of what takes place when a function is called. The best example of this, and where I got the idea from, is this article by Tincho Abbate: https://www.notonlyowner.com/learn/what-happens-when-you-send-one-dai. His article is excellent albeit very long, I'm suggesting something more concise but with a similar concept: explaining most of what happens under the hood in an easy-to-understand way, unifying the entire flow, which should allow for improving intuition.

@critesjosh critesjosh added A-documentation Area: relates to documentation T-feedback Type: recording user feedback labels Dec 21, 2023
@critesjosh critesjosh self-assigned this Dec 21, 2023
@github-project-automation github-project-automation bot moved this to Todo in A3 Dec 21, 2023
critesjosh added a commit that referenced this issue Jan 8, 2024
- Include a high level overview in Foundational Concepts of the
transaction lifecycle.
- formatting the common patterns page
- added a note about a generic error to the Debugging page

Closes #3764 

# Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if
the PR is ready to merge.
- [x] If the pull request requires a cryptography review (e.g.
cryptographic algorithm implementations) I have added the 'crypto' tag.
- [ ] I have reviewed my diff in github, line by line and removed
unexpected formatting changes, testing logs, or commented-out code.
- [x] Every change is related to the PR description.
- [ ] I have
[linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue)
this pull request to relevant issues (if any exist).
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Jan 8, 2024
michaelelliot pushed a commit to Swoir/noir_rs that referenced this issue Feb 28, 2024
- Include a high level overview in Foundational Concepts of the
transaction lifecycle.
- formatting the common patterns page
- added a note about a generic error to the Debugging page

Closes AztecProtocol#3764 

# Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if
the PR is ready to merge.
- [x] If the pull request requires a cryptography review (e.g.
cryptographic algorithm implementations) I have added the 'crypto' tag.
- [ ] I have reviewed my diff in github, line by line and removed
unexpected formatting changes, testing logs, or commented-out code.
- [x] Every change is related to the PR description.
- [ ] I have
[linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue)
this pull request to relevant issues (if any exist).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-documentation Area: relates to documentation T-feedback Type: recording user feedback
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant