Conversation
product-security-plaid-production
bot
requested a review
from fernandezlautaro
|
👋 yashnevatia, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
docs/changelog.md
Outdated
| @@ -0,0 +1,150 @@ | |||
| # Changelog | |||
There was a problem hiding this comment.
I'm assuming that this is supposed to be an example changelog but it looks like a hallucination instead
docs/getting-started.md
Outdated
|
|
||
| ## Quick Start: MultiNode | ||
|
|
||
| The most common entry point is `MultiNode`, which manages connections to multiple RPC endpoints. |
There was a problem hiding this comment.
What is the point of this quick start?
ilija42
left a comment
ilija42
left a comment
There was a problem hiding this comment.
What is the .nojekyll file? Also the MultiNode Component Diagram. Also I don't find most of the diagrams useful. For example I don't really understand what the TXM Component structure is supposed to tell me, what is the point of that External section with HeadTracker, what is the relationship between the components?
| @@ -0,0 +1,10 @@ | |||
| - [Overview](overview.md) | |||
| - [Getting Started](getting-started.md) | |||
There was a problem hiding this comment.
Contributing and changelog don't exist anymore, also not sure if we even need this file
|
|
||
| ## Write Target | ||
|
|
||
| The WriteTarget capability enables workflow-based transaction submission. |
There was a problem hiding this comment.
worth saying that its being deprecated in favour of CRE chain capabilities
There was a problem hiding this comment.
arguably you can even remove this section since we won't be implementing a new write target for any chains
| @@ -0,0 +1,104 @@ | |||
| <!DOCTYPE html> | |||
2 participants
Description
Requires Dependencies
Resolves Dependencies