|
|
|
# Sept 13, 2021
|
|
|
|
----------------------------
|
|
|
|
|
|
|
|
7:04 AM - Started work
|
|
|
|
|
|
|
|
- Cloned work to 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.
|
|
|
|
- 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')
|
|
|
|
StudentPage.<anonymous>
|
|
|
|
C:/Code/Work/feedbackmanager/client/src/student/pages/StudentPage.js:49
|
|
|
|
46 | () => {
|
|
|
|
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>
|
|
|
|
|
|
|
|
4:25pm - Finished Work |
|
|
\ No newline at end of file |
