- Took a bit of time to get the project configured to run on my machine
- 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.
- 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)’
- 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.
from GraphicalInsights.js. This looks like it was a copy-paste mistake
<br>
<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.
> 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>
<br>
from GraphicalInsights.js. This looks like it was a copy-pate mistake.<br><br>
4:25pm - Finished Work
4:25pm - Finished Work
...
@@ -129,13 +128,43 @@ Error thrown on compilation
...
@@ -129,13 +128,43 @@ Error thrown on compilation
### 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 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
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.
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
- 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.
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
3:24 AM - Finished Work
\ No newline at end of file
# 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)
