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

doc update - infographics and overview.md #187

Merged
merged 19 commits into from
Jul 27, 2017
Merged

doc update - infographics and overview.md #187

merged 19 commits into from
Jul 27, 2017

Conversation

rigelrozanski
Copy link
Contributor

No description provided.

infographic basics completed

Update infographic.xml
Update infographic.xml

Update infographic.xml

Update infographic.xml
Added overview-security.png
Added tx.png
Update infographic.xml

Update infographic.xml

Added tx.png

Added tx.png

Update overview.md

Added middleware.png

Added middleware.png

small infograpic doc changes
@@ -0,0 +1,13 @@
# Usage

Graphics can be imported and edited with the draw.io tool:
Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome!


### Transaction (tx)

Each tx passes through the middleware stack which can be defined uniquely by
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice. Thanks for making this clear

@ethanfrey ethanfrey merged commit 9708059 into unstable Jul 27, 2017
@ethanfrey ethanfrey deleted the infographic branch July 27, 2017 19:31
ParthDesai pushed a commit to ChorusOne/cosmos-sdk that referenced this pull request Apr 19, 2021
* Fix 24/getConsensusState

Under `getConsensusState`,

> `getConsensusState` MUST return the consensus state for the consensus algorithm of the host chain at the specified height, for all heights greater than zero and less than or equal to the current height.

For some cases `getConsensusState` will not be able to return the consensus state with height of `0 < _ < current`. For efficiency, the chains might choose to prune the states and (if the pruning happens for past enough states) the impl will still work, so the restriction should be RECOMMENDED.

* Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants