| ... | ... | @@ -85,11 +85,11 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br> |
|
|
|
|
|
|
|
> 1) Bug switching courses as istructor
|
|
|
|
|
|
|
|
##### Issue
|
|
|
|
### Issue
|
|
|
|
|
|
|
|
- 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.
|
|
|
|
- CoursePage.js 195-199
|
| ... | ... | @@ -101,7 +101,7 @@ from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br> |
|
|
|
|
|
|
|
> 2) Mongo Warning
|
|
|
|
|
|
|
|
##### Issue
|
|
|
|
### Issue
|
|
|
|
|
|
|
|
When starting the system a warning shows that needs to be hunted down and fixed.
|
|
|
|
<br>
|
| ... | ... | @@ -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)"
|
|
|
|
```
|
|
|
|
|
|
|
|
##### 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)
|
|
|
|
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 |
|
|
|
|
|
|
|
> 3) Client Side Error StudentFeedback.js
|
|
|
|
|
|
|
|
##### Issue
|
|
|
|
### Issue
|
|
|
|
|
|
|
|
```
|
|
|
|
Error thrown on compilation
|
| ... | ... | @@ -127,7 +127,7 @@ Error thrown on compilation |
|
|
|
[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,
|
|
|
|
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 |
|
|
|
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
|
|
|
|
and all other solutions may clutter the code.
|
|
|
|
|
|
|
|
3:21 AM - Finished Work |
|
|
\ No newline at end of file |
|
|
|
3:23 AM - Finished Work |
|
|
\ No newline at end of file |
