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
feat(cloning): add calls to archive/restore/delete clones #1578
Conversation
… feat/clone-cell-deletion
… feat/clone-cell-deletion
… feat/clone-cell-deletion
… feat/clone-cell-deletion
… feat/clone-cell-deletion
… feat/clone-cell-deletion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice, clean, straightforward, easy to read implementation!
.conductor | ||
.restore_clone_cell(&app_id, &clone_cell_id) | ||
.await?; | ||
self.create_and_add_initialized_cells_for_running_apps(self.clone(), Some(&app_id)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I see you added the line I expected above, here. Yeah, I see why you did this, since it's weird to pass a ConductorHandle down into the Conductor. I think ideally the ConductorHandle shouldn't do anything that the Conductor can't do itself. However, the whole ConductorHandle/Conductor thing has become a mess that needs to be reworked so I'm ok with this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I completely agree
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abe-njama Please add this as a refactoring item for the post-beta phase.
… feat/clone-cell-deletion
… feat/clone-cell-deletion
e170b98
to
92cb1fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done
Co-authored-by: Michael Dougherty <maackle.d@gmail.com>
Summary
document
and clonesdocument.0
,document.1
etc.; this call deletes all clones that have been archived permanently; not undoable, clones cannot be restored)phenotype
tomodifiers
(mostlyDnaPhenotype
toDnaModifiers
)DnaModifiers
in serde for structsAppRoleDnaManifest
andCreateCloneCellPayload
TODO: