@@ -85,11 +85,11 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
...
@@ -85,11 +85,11 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
> 1) Bug switching courses as istructor
> 1) Bug switching courses as istructor
##### Issue
### Issue
- Would get an error when switching courses in the instructors account
- Would get an error when switching courses in the instructors account
##### Fix
### Fix
- Incorrect function definition was used. But the function was the wrong one to use anyway.
- Incorrect function definition was used. But the function was the wrong one to use anyway.
- CoursePage.js 195-199
- CoursePage.js 195-199
...
@@ -101,7 +101,7 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
...
@@ -101,7 +101,7 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
> 2) Mongo Warning
> 2) Mongo Warning
##### Issue
### Issue
When starting the system a warning shows that needs to be hunted down and fixed.
When starting the system a warning shows that needs to be hunted down and fixed.
<br>
<br>
...
@@ -110,7 +110,7 @@ When starting the system a warning shows that needs to be hunted down and fixed.
...
@@ -110,7 +110,7 @@ When starting the system a warning shows that needs to be hunted down and fixed.
[0] (Use node --trace-warnings ... to show where the warning was created)"
[0] (Use node --trace-warnings ... to show where the warning was created)"
```
```
##### Fix
### Fix
- According to [this](https://www.mongodb.com/community/forums/t/warning-accessing-non-existent-property-mongoerror-of-module-exports-inside-circular-dependency/15411/6)
- According to [this](https://www.mongodb.com/community/forums/t/warning-accessing-non-existent-property-mongoerror-of-module-exports-inside-circular-dependency/15411/6)
forum, a MongoDb employee claims its a problem in the Mongo code itself. She claims that the Node driver team at Mongo has been
forum, a MongoDb employee claims its a problem in the Mongo code itself. She claims that the Node driver team at Mongo has been
...
@@ -118,7 +118,7 @@ made aware of the situation and this it is "safe to ignore and will hopefully be
...
@@ -118,7 +118,7 @@ made aware of the situation and this it is "safe to ignore and will hopefully be
> 3) Client Side Error StudentFeedback.js
> 3) Client Side Error StudentFeedback.js
##### Issue
### Issue
```
```
Error thrown on compilation
Error thrown on compilation
...
@@ -127,7 +127,7 @@ Error thrown on compilation
...
@@ -127,7 +127,7 @@ Error thrown on compilation
[1] Line 42:21: 'key' is assigned a value but never used
[1] Line 42:21: 'key' is assigned a value but never used
````
````
##### Fix
### Fix
- In function handleCSVDownloas, rather than `var data = [];` I expanded the scope to by initializing as a `let data = [];` as when initializing with a var,
- In function handleCSVDownloas, rather than `var data = [];` I expanded the scope to by initializing as a `let data = [];` as when initializing with a var,
the valueof data becomes inconsistent when it enters the if statement and comes back out. A let ensures that `data` is the same
the valueof data becomes inconsistent when it enters the if statement and comes back out. A let ensures that `data` is the same
...
@@ -138,4 +138,4 @@ a console.log test, I got different results. Plus, logically, there is a big dif
...
@@ -138,4 +138,4 @@ a console.log test, I got different results. Plus, logically, there is a big dif
I don't want to risk making the code dangerous. It's not the greatest fix, but keep in mind that this warning is basically harmless
I don't want to risk making the code dangerous. It's not the greatest fix, but keep in mind that this warning is basically harmless
