Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
ADR 35: Rosetta API Support #7492
ADR 35: Rosetta API Support #7492
Changes from 94 commits
b21ce41
969c4c9
a46fbeb
1ee3307
95eabea
52493b6
ebe39fb
30380bf
5afa87e
b6f3885
dc35d65
3f38486
52c5ed9
b4f6d63
8943388
a879351
2c5241e
6b8d899
2d34cb7
818c5bf
b94d785
b61dafb
7921d5c
2f4836c
33b35bd
d784100
987addc
a0562d2
26e3e67
91602d6
4cef432
1eff674
244150f
be43346
2cb2ef5
436aeb8
e86321e
6316058
45c5f8e
b50eec1
9e0c956
16239c4
f9a362d
d95157c
2bc1c4f
60ed224
bb874b4
1b86983
cb9062e
942daf0
e73bb03
8e06e7c
39e5178
bc244fc
e52a693
609aab3
91e7199
ce2e7e9
3e4674a
eca8545
8b9371e
e94e337
7f01ec0
79786dc
ac49826
772ef08
ca2b7d3
31a7e1a
2816b06
387a0bd
3ab45fd
af5ae37
f992a96
64cce9d
ccf5385
8e3ffa4
231530f
5407ba3
b63c6e9
7802404
d663270
d207329
a9c0de7
0bc99ed
573b3c3
22efc7e
3ea1a2a
82c632d
9cd9af0
40ba26b
eb5cbf4
6d986a3
db073c4
73793ef
e961cab
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
This diverges from the constructor defined above
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.
@jgimeno bump
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.
@aaronc True that, yet this is an example, i.e. the actual code would diverge slightly between 0.39 and 0.40 - as described in the comment above:
each Cosmos SDK release series will have version specific implementations of Network and Adapter, as well as a NewNetwork constructor.
For instance, that's the signature of this very constructor for Stargate:
The constructor's signature for Launchpad follows:
I hope this helps.
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.
Is there an example of the adapter that should be linked to in this ADR for reference?
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.
@jgimeno bump
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.
We can use the one for launchpad, but is in a branch and not merged yet. Is that ok?
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.
Yes, go ahead @jgimeno. Use a permalink
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.
cosmos-sdk/server/rosetta/launchpad.go
Line 75 in eeb4c53
This is the adapter constructor in the launchpad implementation, but the implementation itself is based in several files.
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.
The team is finalising the Stargate's
Adapter
implementation. I think we can show some simplified code example here?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 does not have to be complete - this is an ADR after all, not a code walkthrough. Either some pseudo code or a code snippet should it.