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.

Sign up for GitHub

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

Jump to bottom

docs: updating roadmap #877

Merged
merged 3 commits into from
Sep 23, 2024
Merged

docs: updating roadmap #877

merged 3 commits into from
Sep 23, 2024

Conversation

Dominik1999
Copy link
Collaborator

This PR updates the roadmap. I push against main because we build the docs from main

bobbinth
bobbinth reviewed Sep 17, 2024
View reviewed changes
Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

Thank you! A few comments:

  1. "Rate limits" should go into "Developer experience".
  2. "Efficient ECDSA signatures" can also probably go into "Developer experience" or at least i"m not sure this needs to be in "Core protocol"
  3. Let's move "Block kernel" and "Batch kernel" into separate boxes.

@bobbinth
Copy link
Contributor

A couple more comments:

  • Core protocol
    • I would move "transaction recency conditions" and "foreign procedure invocation" to the top of the box. This way, they'll be closer to Oracles/Rate limits.
  • ZK L2
    • Block kernel should depend on batch kernel.

bobbinth
bobbinth approved these changes Sep 23, 2024
View reviewed changes
Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you! I'm not sure we need to update the changelog for this PR - but fine to keep it as well.

@Dominik1999 Dominik1999 merged commit 3c5e0fe into main Sep 23, 2024
8 checks passed
@Dominik1999 Dominik1999 deleted the dominik_update_roadmap branch September 23, 2024 06:34
@PhilippGackstatter PhilippGackstatter mentioned this pull request Sep 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bobbinth bobbinth bobbinth approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Dominik1999 @bobbinth