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

Dump full state, importing lots of types from https://github.com/holochain-open-dev/core-types #94

Closed
wants to merge 18 commits into from

Conversation

guillemcordoba
Copy link
Contributor

In the effort to make the Holochain Playground compatible with a real Holochain conductor, I've had to create all the types that the dump_full_state function is returning. I thought it would be useful to include them at the @holochain/conductor-api level, so that everyone gets them. Let me know if you'd prefer to keep them at https://github.com/holochain-open-dev/core-types.

@guillemcordoba guillemcordoba requested a review from zippy December 6, 2021 09:55
@Connoropolous
Copy link
Contributor

@guillemcordoba this still under consideration for review?

@guillemcordoba
Copy link
Contributor Author

Yeap!

Copy link
Contributor

@jost-s jost-s left a comment

Choose a reason for hiding this comment

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

This is brilliant, thanks so much @guillemcordoba !!!

I've included this work into the new repo of the JS client.

@jost-s
Copy link
Contributor

jost-s commented Jan 12, 2022

I'm closing this PR regardless, because we're archiving the repo in favor of the new one.

@jost-s jost-s closed this Jan 12, 2022
@jost-s
Copy link
Contributor

jost-s commented Jan 13, 2022

My mistake, we're not archiving this repo but only changing its name. Regardless, this branch's changes are included.

@jost-s jost-s deleted the dump-full-state branch January 13, 2022 09:37
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.

3 participants