from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
4:25pm - Finished Work
8:05 AM - started work
# Sept 14, 2021
## Overview
- Fixed some bugs
- Spent most of the time trying to get my head around the server side of things. Yesterday I spent most of the time on the client side.
- Having a bit of difficulty understanding the endpoints. It's taking a while but I'll keep at it. What can I say, hard work pays off 1 Timothy 2:6.
- Was going to work on parsing the epics into issues, but I need to figure out my game plan for developing the individual repo feature
- I think tomorrow I'll be spending some more time in the server side of things, after that, I should have enough of an understanding to start implementing the individual repo feature
- I also did some reading on the Gitlab api, and other api's of dependencies I haven't encountered yet.
- It's more of a learning day today, but hopefully I can make some actual progress by the end of the week.
- Going to need to create my goals for this co-op job soon, I'll probably have something about enhancing my server side programming skills.
## Bug Fixes
> 1) Bug switching courses as istructor
<br>
##### Issue
- Would get an error when switching courses in the instructors account
##### Fix
- Incorrect function definition was used. But the function was the wrong one to use anyway.
- CoursePage.js 195-199
- Select Course (done on button click which runs this function)
- After course select, run getProjectsFromServer() to fetch projects for selected course
- After that, run getStudentSummaryDataFromServer() to fetch number of students in the course
- After that, run getStudentFeedbackFromServer to get the student feedback data (numbers and graphical representations)
> 2) Mongo Warning
<br>
##### Issue
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
- 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
made aware of the situation and this it is "safe to ignore and will hopefully be gone in upcoming releases".
> 3) Client Side Error StudentFeedback.js
<br>
##### Issue
```
Error thrown on compilation
./src/instructor/components/StudentFeedback.js
[1] Line 33:48: Function declared in a loop contains unsafe references to variable(s) 'data' no-loop-func
[1] Line 42:21: 'key' is assigned a value but never used
````
##### 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
variable being modified both in the if statement and out of it.
- Since it was complaining about key not being usedon line 42, i simple disabled the warning for that line by setting the
comment `//eslint-disable-next-line no-unused-vars` above it. I tried setting the variable to `const [value]` however, when I did
a console.log test, I got different results. Plus, logically, there is a big difference between a key value pair and an array therefore
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
