Tim's-Journal.md

@@ -7,8 +7,7 @@
 - Took a bit of time to get the project configured to run on my machine
 - To run on my machine, get rid of ‘BROWSER’ Variable from /client/package.lock.
 - Unable to use .env, must have a name before the extension on windows (ex. a.env)’
-<br><br>            
-I spent most of my time reading through the code. Thus, I did very little coding other than playig around with some thing. I think I know what I'm doing.
+- I spent most of my time reading through the code. Thus, I did very little coding other than playig around with some thing. I think I know what I'm doing.
 
 ## Bugs
 
@@ -58,11 +57,11 @@ const FormContainer = styled.div`
   background-color: white;
 `;
 ```
+from GraphicalInsights.js. This looks like it was a copy-paste mistake
 <br>
 
 > 3) Potetial bug is that Not all projects are shown for C_Course_Test for s5test student. I don't think the server is retrieving all the data.
 <br>
-from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
 
 4:25pm - Finished Work
 
@@ -129,13 +128,43 @@ Error thrown on compilation
 
 ### 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 
+- In function handleCSVDownloas, rather than `var data = [];` I expanded the scope by initializing as a `let data = [];` because when initializing with a var, the value of 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.
+- Since it was complaining about key not being used on line 42, I simply 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:24 AM - Finished Work
+
+# Sept 15, 2021
+
+8:15 AM - Started work
+
+- Spent most of the day in the client/src/student/components folder.
+- Wanting to familiarize myself with the student side to be able to effeciently implement the individual repo feature.
+- Came up with 2 UI ideas for implementing a "add project" button.
+- did a deep dive in React api 
+
+1:36 PM - stepped away for a bit.<br>
+2:09 PM - restarted working
+
+## Features
+
+### Add Repo Button
+
+- I already Incorperated an Add Repo button to the student page. It is of a similar style to the one found in the instructor page
+- This button currently does nothing.
+- Button will eventually redirect to a page which will ask the student which project they want to incorperate into their IFS account
+- Will make a branch for this feature in a bit.
+
+### Add Repo Game Plan
+
+- Each repository has a unique `project ID`, which is different from its `group ID`.
+- I am going to create a new `addRepoPage` which will ask the student to input the project ID of the repo they want to add
+- Naturally, the Database for these repos will have a `projectID` value functioning similar to the `groupID` value in instructorConfiguration db
+
+### Add New Repo Page
+
+- I did not start on this yet, but for the sake of uniformity, it will have the same aethetic as the `NewCoursePage.js`
+- The page will ask the user to input the project id of the repo they want to add.
+- There should also be a help button to tell the student how to find the projectID (but this isn't necessary)
+
+3:49 PM - Done work
\ No newline at end of file