add action/reason required value checks (#80)

* add new older/outlier combo reason
* fixed bug #77

Author: larrybabb

README.md

@@ -9,6 +9,8 @@ The chrome extension is used by the ClinGen curation workflow (link) ...
 
 ### v1.13 changes
 * Added new combo flagging candidate reason 'Older and Outlier claim with insufficient supporting evidence'.
+* Added requirement check that an action is selected.
+* Added requirement check that a reason is selected for all Flagging Candidates.
 
 ### v1.12.2 bug fix
 * Patched a bug causing the inability to read the SCV data in the Submitter column. This was introduced by a change in the underlying ClinVar UI during the 2/21/2024 release.

scvc/popup.js

@@ -64,6 +64,16 @@ window.addEventListener('DOMContentLoaded', () => {
         return;
       }
 
+      if (!data.action) {
+        alert("An action is required. Please select one from the dropdown before submitting.");
+        return;
+      }
+
+      if (data.action = "Flagging Candidate" && !data.reason) {
+        alert("A reason is required for Flagging Candidates. Please select one from the dropdown before submitting.");
+        return;
+      }
+
       chrome.runtime.sendMessage(data, function(response) {
         var lastError = chrome.runtime.lastError;
         if (lastError) {