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

migrate from gRPC to http (WIP) #73

Merged
merged 64 commits into from
Oct 5, 2020
Merged

Conversation

dckc
Copy link
Collaborator

@dckc dckc commented Jan 26, 2020

this is just a tiny start; it just does status.

note that protobuf DeployDataProto is still needed

ref:
https://github.com/rchain/rchain/blob/ca9a3bc5f7b4332899968e2d5320698c9b6e07db/integration-tests/test/http_client.py

dckc added 30 commits April 20, 2019 08:21
where an interface such as `IDeployData` has:
  deployer?: Uint8Array | null;
the class will have:
  deployer: Uint8Array;

so flow complains about the mismatch. Solution: mark the field
in the interface covariant:
  +deployer?: Uint8Array | null;
flowgen generates "CLASS mixins IFACE"; I don't see that documented
anywhere. "CLASS implements IFACE" seems to work.
avoid "cannot use value as type" or some such
allow .js in imports for browser context
from liquid-democracy Sep 13 c406f2da0ac
 - update blake2 dep
 - package.json lint for scripts
dckc added 29 commits September 23, 2020 23:53
 - fix rnode-openapi-schama types
  - update rhopm to use proxy APIs:
    - update proxy to provide checkForDeployInfo, listenAtDeployId,
    - export makeAccount (works in liquid-democracy Jakefile)
      - replace signWithKey
    - DataForDeploy type is gone
    - clock() is async (no reading the clock twice in the same turn!)
  - export makeConnection (untested)
  - hoist readFileSync out of shardIO
yarn complained about 0.9.25.4
@dckc dckc merged commit 6454e91 into rchain-community:master Oct 5, 2020
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.

1 participant