- 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.
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)
