Unpublish workspaces on deployment

1. +++ b/deploy.links.task.yml
   @@ -8,11 +8,6 @@ entity.replication.canonical:
   -entity.replication.edit_form:
   
   +++ b/deploy.module
   @@ -59,12 +59,10 @@ function deploy_entity_type_alter(array &$entity_types) {
   -      'edit' => 'Drupal\deploy\Entity\Form\ReplicationForm',
   ...
   -    $replication->setLinkTemplate('edit-form', '/admin/structure/deployment/{replication}/edit');
   

   I was confused by this at first, but it doesn't make sense to be able to edit a replication!

   +1

2. +++ b/src/Entity/Form/ReplicationForm.php
   @@ -107,6 +108,15 @@ class ReplicationForm extends ContentEntityForm {
   +        '#default_value' => FALSE,
   

   I'm undecided on the default, maybe worth checking with Dick, Jozef, and Luca?

3. +++ b/src/Form/ReplicationActionForm.php
   @@ -125,10 +126,16 @@ class ReplicationActionForm extends FormBase {
   +      $form['message'] = $this->generateMessageRenderArray('warning', 'This deployment cannot be re-deployed because source or target workspace has been archived.');
   

   Might be nice if we can work out which workspace has been archived?

1. +++ b/src/Entity/Form/ReplicationForm.php
   @@ -107,6 +109,15 @@ class ReplicationForm extends ContentEntityForm {
   +    if ($source_workspace->id() != \Drupal::getContainer()->getParameter('workspace.default')) {
   

   I added a new method in Multiversion so you could now just do !$source_workspace->isDefaultWorkspace() here.

2. +++ b/src/Entity/Form/ReplicationForm.php
   @@ -107,6 +109,15 @@ class ReplicationForm extends ContentEntityForm {
   +        '#description' => $this->t('If selected and deployment finish with success, the workspace will be archived.'),
   

   The wording here doesn't quite sound right. Maybe, "The workspace will be archived on successful deployment."?

3. +++ b/src/Entity/Form/ReplicationForm.php
   @@ -156,7 +167,26 @@ class ReplicationForm extends ContentEntityForm {
   +        if ($form_state->hasValue('archive') && $form_state->getValue('archive') == TRUE) {
   

   We can use === TRUE here.

ok, looks good!