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

refactor(clone-cell)!: return cell info for clone operations #1763

Merged
merged 21 commits into from
Jan 19, 2023

Conversation

jost-s
Copy link
Contributor

@jost-s jost-s commented Jan 14, 2023

Summary

  • CreateCloneCell returns CellInfo instead of InstalledCell.
  • EnableCloneCell returns CellInfo instead of InstalledCell.
  • CellInfo variants are renamed to snake case during serde.
  • Remove unimplemented app request SignalSubscription.

Fixes holochain/holochain-client-js#161

TODO:

  • CHANGELOG(s) updated with appropriate info
  • Just before pressing the merge button, ensure new entries to CHANGELOG(s) are still under the UNRELEASED heading

@jost-s jost-s requested a review from maackle January 14, 2023 17:46
Copy link
Collaborator

@maackle maackle left a comment

Choose a reason for hiding this comment

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

This looks good after what I posted is addressed

crates/holochain/src/conductor/conductor.rs Outdated Show resolved Hide resolved
crates/holochain/src/conductor/conductor.rs Outdated Show resolved Hide resolved
@jost-s jost-s requested a review from maackle January 17, 2023 23:12
jost-s and others added 7 commits January 17, 2023 20:29
* Write failing test for disabling clones

* Fix problem where disabled clones are started at conductor restart

* Fixes for macos (#1777)

* chore: update holonix version

* chore(nix):fix dev shell for macos

* chore(kitsune_p2p/types): update sysinfo to fix build

* chore(nix): update holonix version

* chore(nix): remove crate2nix override

* chore(nix): revert disabling crate2nix for macos

* chore(nix): update holonix

* dummy commit to trigger CI

* chore(nix): use crate2nix from nixpkgs

* docs(readme): remove chat + forum and add Discord link (#1778)

* Add Discord link to Readme

Removed the link to chat.holochain.net and in its place added the discord server link to the readme

* Update README.md

* countersigning top level docs (#1768)

* countersigning top level docs

* Update crates/hdk/src/lib.rs

Co-authored-by: Jost Schulte <[email protected]>

* Update crates/hdk/src/lib.rs

Co-authored-by: Jost Schulte <[email protected]>

Co-authored-by: Jost Schulte <[email protected]>

* fix(app-install): reject duplicate identical cells (#1773)

* add failing test for two identical apps install

* add failing test for duplicate app installation

* check there's no identical running cell when installng an app

* update changelog

* search through all cells for identical cell

* add specific error for duplicate cells

* clippy: combine find & is_some() to any

* document call_info in hdk (#1765)

* Apply suggestions from code review

Co-authored-by: Jost Schulte <[email protected]>

* remove docs saying links are to entries (#1767)

Co-authored-by: DavHau <[email protected]>
Co-authored-by: Abraham Njama <[email protected]>
Co-authored-by: David Meister <[email protected]>
Co-authored-by: Jost Schulte <[email protected]>
@maackle maackle enabled auto-merge (squash) January 19, 2023 20:03
@maackle maackle merged commit 0737cf3 into develop Jan 19, 2023
@maackle maackle deleted the refactor/create-clone-cell-returns-cell-info branch January 19, 2023 21:11
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.

Cell Info design for multi instance use on the client
3 participants