when force-sending SCORM data to the fallback, use cmi.completion_sta…

…tus for `completed`

fixes #300

I also noticed that the message "Attempt data is being saved to the database. Please wait."
is shown after the attempt is ended due to passing the "available until"
time. This commit changes it so that text is not shown when the attempt
is automatically ended.

numbas_lti/static/api.js

@@ -672,7 +672,7 @@ SCORM_API.prototype = {
             },
             body: JSON.stringify({
                 batches: batches,
-                complete: force
+                complete: this.data['cmi.completion_status']=='completed'
             })
         });
 

numbas_lti/static/run_attempt.css

@@ -39,7 +39,7 @@ body.terminated #status-display .symbol, #status-display .symbol {
 #status-display.ok .text-ok,
 #status-display.confirmation .text-confirmation,
 #status-display.unavailable-at .text-unavailable-at,
-#status-display.terminated:not(.ok):not(.failed-final-ajax) .text-terminated,
+#status-display.terminated:not(.ok):not(.failed-final-ajax):not(.ended) .text-terminated,
 #status-display.failed-final-ajax .text-failed-final-ajax,
 #status-display.loading-error .text-loading-error
 {