- 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.
## Bugs
### Discovered
> 1) If a student logs in and has no initial courses in his account, it will crash<br>
```
TypeError: Cannot read properties of undefined (reading 'id')
47 | //Presently initializing the selected course to be the first in the array received from the server
48 | this.setState({
> 49 | selectedCourse: this.state.courses[0].id,
| ^ 50 | });
51 | }
52 | );
```
I have fixed this bug by inserting the following code:
```
//Presently initializing the selected course to be the first in the array received from the server
if(this.state.courses[0]) { // TODO - eventually, courses[0] will be an 'add course' option if student has no initial courses so we can get rid of this then\
this.setState({
selectedCourse: this.state.courses[0].id,
});
}
```
<br>
> 2) Probably the same as 1) but this time in CoursePage.js<br>
### Fixed
> 1) fixed the one in discovered above<br>
> 2) fixed the GraphiclInsights bug<br>
I removed the following code:<br>
```
const FormContainer = styled.div`
border-radius: 5px;
padding: 15px;
background-color: white;
`;
```
<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>