FIX: straighten-out wizard-level subscription features

assets/javascripts/discourse/templates/admin-wizards-wizard-show.hbs

@@ -55,21 +55,11 @@
   <div class="wizard-settings">
     <div class="setting">
       <div class="setting-label">
-        <label>{{i18n "admin.wizard.required"}}</label>
+        <label>{{i18n "admin.wizard.save_submissions"}}</label>
       </div>
       <div class="setting-value">
-        {{input type="checkbox" checked=wizard.required}}
+        {{input type="checkbox" checked=wizard.save_submissions}}
-        <span>{{i18n "admin.wizard.required_label"}}</span>
+        <span>{{i18n "admin.wizard.save_submissions_label"}}</span>
-      </div>
-    </div>
-
-    <div class="setting">
-      <div class="setting-label">
-        <label>{{i18n "admin.wizard.after_signup"}}</label>
-      </div>
-      <div class="setting-value">
-        {{input type="checkbox" checked=wizard.after_signup}}
-        <span>{{i18n "admin.wizard.after_signup_label"}}</span>
       </div>
     </div>
 
@@ -83,6 +73,16 @@
       </div>
     </div>
 
+    <div class="setting">
+      <div class="setting-label">
+        <label>{{i18n "admin.wizard.after_signup"}}</label>
+      </div>
+      <div class="setting-value">
+        {{input type="checkbox" checked=wizard.after_signup}}
+        <span>{{i18n "admin.wizard.after_signup_label"}}</span>
+      </div>
+    </div>
+
     <div class="setting">
       <div class="setting-label">
         <label>{{i18n "admin.wizard.prompt_completion"}}</label>
@@ -108,32 +108,14 @@
       </div>
     </div>
 
-    <div class="setting full field-mapper-setting">
-      <div class="setting-label">
-        <label>{{i18n "admin.wizard.permitted"}}</label>
-      </div>
-      <div class="setting-value">
-        {{wizard-mapper
-          inputs=wizard.permitted
-          options=(hash
-            context="wizard"
-            inputTypes="assignment,validation"
-            groupSelection="output"
-            userFieldSelection="key"
-            textSelection="value"
-            inputConnector="and"
-          )}}
-      </div>
-    </div>
-
     {{#wizard-subscription-container}}
       <div class="setting">
         <div class="setting-label">
-          <label>{{i18n "admin.wizard.save_submissions"}}</label>
+          <label>{{i18n "admin.wizard.required"}}</label>
         </div>
         <div class="setting-value">
-          {{input type="checkbox" checked=wizard.save_submissions}}
+          {{input type="checkbox" checked=wizard.required}}
-          <span>{{i18n "admin.wizard.save_submissions_label"}}</span>
+          <span>{{i18n "admin.wizard.required_label"}}</span>
         </div>
       </div>
 
@@ -146,6 +128,24 @@
           <span>{{i18n "admin.wizard.restart_on_revisit_label"}}</span>
         </div>
       </div>
+
+      <div class="setting full field-mapper-setting">
+        <div class="setting-label">
+          <label>{{i18n "admin.wizard.permitted"}}</label>
+        </div>
+        <div class="setting-value">
+          {{wizard-mapper
+            inputs=wizard.permitted
+            options=(hash
+              context="wizard"
+              inputTypes="assignment,validation"
+              groupSelection="output"
+              userFieldSelection="key"
+              textSelection="value"
+              inputConnector="and"
+            )}}
+        </div>
+      </div>
     {{/wizard-subscription-container}}
   </div>
 

assets/stylesheets/common/admin.scss

@@ -183,16 +183,6 @@ $error: #ef1700;
   @extend .wizard-settings-group;
 }
 
-.admin-wizard-container.settings {
-  .wizard-settings {
-    .wizard-subscription-container {
-      [class~="setting"] {
-        margin-bottom: 0;
-      }
-    }
-  }
-}
-
 .wizard-custom-field {
   background: transparent;
   background-color: var(--primary-very-low);
@@ -868,7 +858,7 @@ $error: #ef1700;
   .subscription-header {
     display: flex;
     justify-content: space-between;
-    margin-bottom: 0.25em;
+    margin-bottom: 0.5em;
 
     h3 {
       margin: 0;

lib/custom_wizard/subscription.rb

@@ -18,6 +18,12 @@ class CustomWizard::Subscription
           standard: ['*'],
           business: ['*'],
           community: ['*']
+        },
+        restart_on_revisit: {
+          none: [],
+          standard: ['*'],
+          business: ['*'],
+          community: ['*']
         }
       },
       step: {