Skip to content

BB-1848: Fix bug on status pill in the first deployment

Giovanni Cimolin da Silva requested to merge giovanni/fix-small-bug into master

This PR fixes a small bug in the status pill that shows the wrong deployment message in the first deployment.

Testing instructions:

  1. Use the master branch.
  2. Go to the registration form (http://localhost:3000) and create a new account (don't forget to activate the email).
  3. Apply this diff to make testing easier:
diff --git a/registration/api/v2/views.py b/registration/api/v2/views.py
index 81fc8c2d..d36e54ed 100644
--- a/registration/api/v2/views.py
+++ b/registration/api/v2/views.py
@@ -547,7 +547,7 @@ class OpenEdxInstanceDeploymentViewSet(CreateAPIView, RetrieveDestroyAPIView, Ge
         data = {
             'undeployed_changes': undeployed_changes,
             'deployed_changes': deployed_changes,
-            'status': deployment_status.name,
+            'status': DeploymentState.provisioning.name,
             'deployment_type': deployment_type,
         }
  1. Check that the status pill says deploying instead of preparing instance.
  2. Checkout this branch.
  3. Check that the status pill looks like this: image

Merge request reports