Tim's-Journal.md

@@ -65,3 +65,80 @@ const FormContainer = styled.div`
 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.
+<br>
+```
+"(node:38032) Warning: Accessing non-existent property 'MongoError' of module exports inside circular dependency
+[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
+and all other solutions may clutter the code.
+
+3:21 AM - Finished Work
\ No newline at end of file