Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Problem: tdbe types are not up to date with latest design (fixes #1967) #1977

Merged
merged 1 commit into from
Jul 23, 2020

Conversation

tomtau
Copy link
Contributor

@tomtau tomtau commented Jul 22, 2020

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

  • chain-abci part was moved to extras module of mls for validation
    (when Mainnet docs Seeds Update crypto-org-chain/chain-docs#141 is done
    implementation could go there)
  • in order not to break tx format, confidential init is still takes as Vec
  • as tdbe workflows aren't ready yet, a temporary wrapper was put in the client

@tomtau
Copy link
Contributor Author

tomtau commented Jul 22, 2020

bors r+

bors bot added a commit that referenced this pull request Jul 22, 2020
1977: Problem: tdbe types are not up to date with latest design (fixes #1967) r=tomtau a=tomtau

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

- chain-abci part was moved to extras module of mls for validation
(when crypto-org-chain/chain-docs#141 is done
implementation could go there)
- in order not to break tx format, confidential init is still takes as Vec<u8>
- as tdbe workflows aren't ready yet, a temporary wrapper was put in the client


Co-authored-by: Tomas Tauber <[email protected]>
@tomtau tomtau force-pushed the update/tdbe-types branch from f9d96d1 to 169e9bb Compare July 22, 2020 07:52
@tomtau
Copy link
Contributor Author

tomtau commented Jul 22, 2020

bors r-

@tomtau
Copy link
Contributor Author

tomtau commented Jul 22, 2020

bors r+

bors bot added a commit that referenced this pull request Jul 22, 2020
1977: Problem: tdbe types are not up to date with latest design (fixes #1967) r=tomtau a=tomtau

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

- chain-abci part was moved to extras module of mls for validation
(when crypto-org-chain/chain-docs#141 is done
implementation could go there)
- in order not to break tx format, confidential init is still takes as Vec<u8>
- as tdbe workflows aren't ready yet, a temporary wrapper was put in the client


Co-authored-by: Tomas Tauber <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jul 22, 2020

Build failed:

@tomtau tomtau force-pushed the update/tdbe-types branch from 169e9bb to 5a3a3d9 Compare July 22, 2020 10:22
@tomtau
Copy link
Contributor Author

tomtau commented Jul 22, 2020

bors r+

bors bot added a commit that referenced this pull request Jul 22, 2020
1977: Problem: tdbe types are not up to date with latest design (fixes #1967) r=tomtau a=tomtau

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

- chain-abci part was moved to extras module of mls for validation
(when crypto-org-chain/chain-docs#141 is done
implementation could go there)
- in order not to break tx format, confidential init is still takes as Vec<u8>
- as tdbe workflows aren't ready yet, a temporary wrapper was put in the client


Co-authored-by: Tomas Tauber <[email protected]>
Copy link
Collaborator

@leejw51crypto leejw51crypto left a comment

Choose a reason for hiding this comment

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

lgtm

@tomtau tomtau force-pushed the update/tdbe-types branch 2 times, most recently from a5a5b2d to 961b12d Compare July 22, 2020 13:17
@tomtau
Copy link
Contributor Author

tomtau commented Jul 22, 2020

bors r+

bors bot added a commit that referenced this pull request Jul 22, 2020
1977: Problem: tdbe types are not up to date with latest design (fixes #1967) r=tomtau a=tomtau

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

- chain-abci part was moved to extras module of mls for validation
(when crypto-org-chain/chain-docs#141 is done
implementation could go there)
- in order not to break tx format, confidential init is still takes as Vec<u8>
- as tdbe workflows aren't ready yet, a temporary wrapper was put in the client


Co-authored-by: Tomas Tauber <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jul 22, 2020

Build failed:

…to-com#1967)

Solution: updated the data types according to the latest
doc https://github.com/crypto-com/chain-docs/pull/179/files

- chain-abci part was moved to extras module of mls for validation
(when crypto-org-chain/chain-docs#141 is done
implementation could go there)
- in order not to break tx format, confidential init is still takes as Vec<u8>
- as tdbe workflows aren't ready yet, a temporary wrapper was put in the client
@tomtau tomtau force-pushed the update/tdbe-types branch from 961b12d to f0c4528 Compare July 23, 2020 00:40
@tomtau
Copy link
Contributor Author

tomtau commented Jul 23, 2020

bors retry

@bors
Copy link
Contributor

bors bot commented Jul 23, 2020

Build succeeded:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants