Skip to content
This repository was archived by the owner on Jun 7, 2023. It is now read-only.

Commit 5864bb1

Browse files
authored
Merge pull request #1101 from bjones1/fitb_fix
Fix: Remove unnecessary async; fix refactoring bugs.
2 parents 73aa01f + a9801a2 commit 5864bb1

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

runestone/common/js/runestonebase.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,19 +57,18 @@ export default class RunestoneBase {
5757
}
5858
}
5959

60-
async logBookEvent(eventInfo) {
60+
logBookEvent(eventInfo) {
6161
if (this.graderactive) {
6262
return;
6363
}
64-
let post_return = Promise.resolve();
6564
eventInfo.course = eBookConfig.course;
6665
eventInfo.clientLoginStatus = eBookConfig.isLoggedIn;
6766
eventInfo.timezoneoffset = new Date().getTimezoneOffset() / 60;
6867
if (this.percent) {
6968
eventInfo.percent = this.percent;
7069
}
7170
if (eBookConfig.useRunestoneServices && eBookConfig.logLevel > 0) {
72-
post_return = jQuery.post(
71+
var post_return = jQuery.post(
7372
eBookConfig.ajaxURL + "hsblog",
7473
eventInfo,
7574
function (jsondata) {

runestone/fitb/js/fitb.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,6 +245,7 @@ export default class FITB extends RunestoneBase {
245245
let ret = this.logCurrentAnswer();
246246
if (!this.feedbackArray) {
247247
// On success, update the feedback from the server's grade.
248+
let answer = JSON.stringify(this.given_arr);
248249
let that = this;
249250
ret.done(function (data) {
250251
that.setLocalStorage({

0 commit comments

Comments
 (0)