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

use VersionedSlate for the owner_rpc api #202

Merged
merged 1 commit into from
Aug 6, 2019

Conversation

antiochp
Copy link
Member

@antiochp antiochp commented Aug 5, 2019

Resolves #201

Takes the same approach as the foreign_rpc api where we use VersionedSlate on the rpc side of things, converting to Slate internally.
These both serialize/deserialize in an identical fashion currently so this does not affect anything.
But once we start modifying Slate itself (via the contained Transaction) we should be decoupled from these changes.

@antiochp antiochp added the enhancement New feature or request label Aug 5, 2019
@antiochp antiochp requested a review from yeastplume August 5, 2019 12:04
@yeastplume
Copy link
Member

All fine, will merge on top of my other PR and then change the new set of 'secure' functions to use versioned slate as well.

@yeastplume yeastplume merged commit 5cdd412 into mimblewimble:master Aug 6, 2019
@antiochp antiochp deleted the owner_api_versioned_slate branch August 7, 2019 11:41
garyyu pushed a commit to garyyu/grin-wallet that referenced this pull request Sep 5, 2019
garyyu added a commit to gottstech/grin-wallet that referenced this pull request Sep 6, 2019
* Update grin crates to v2.0.4

* rustfmt

* use VersionedSlate for the owner_rpc api (mimblewimble#202)

* rustfmt

* rework kernel features support to handle fee and lock_height on features variants

* translate to/from TransactionV2 correctly in owner api and foreign api

* rustfmt

* revert CbData changes
mining node calls build_coinbase and this needs to be consistent with grin node

* cleanup imports

* fixup controller tests

* rustfmt

* fix the building
yyangli pushed a commit to mwcproject/mwc-wallet that referenced this pull request May 13, 2020
antiochp added a commit to antiochp/grin-wallet that referenced this pull request Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Owner API should leverage VersionedSlate and not Slate directly
2 participants