Merge pull request #36 from OSC/hiding-widgets

use OnDemands hide functionality, and set the min to 1
OSC · Dec 21, 2021 · 13d3cb9 · 13d3cb9
2 parents 238298a + 9f05d3c
commit 13d3cb9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 53 deletions.
diff --git a/form.js b/form.js
diff --git a/form.yml.erb b/form.yml.erb
@@ -26,8 +26,8 @@ attributes:
     help: |
       Number of cores on node type (4 GB per core unless requesting whole
       node). Leave blank if requesting full node.
-    min: 0  # This is treated as requesting the max allowable
-    max: 28  # This will be updated by form.js
+    min: 1 
+    max: 28
     step: 1
   bc_num_slots:
     help: |
@@ -95,7 +95,11 @@ attributes:
     label: "License provider"
     help: "Some users are required to provide their own license information"
     options:
-      - ['I can use the OSC license', 'osc']
+      - [
+          'I can use the OSC license', 'osc',
+          data-hide-extern-license-server: true,
+          data-hide-extern-license-file: true,
+        ]
       - ['I need to use an external license', 'external']
   extern_license_server:
     label: "External License Server"