Removed errors from development console

[rizwan2000rm]

Issue Number

fixes #88

Describe the changes you've made

Removed all the development console errors. Turns out most of them were from a single line as a whole component was passed inside a tag which resulted in many errors.

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)

Failed prop type: Material-UI: This elevation 4 is not implemented

The error above is caused by soft Shadows in Material UI theming options. I have commented it out for now, as I don't understand what those options means maybe someone can help me with that.

Additional context

There are still 2 errors in the console which are due to some dependencies installed.

[Deprecation] SharedArrayBuffer will require cross-origin isolation as of M91, around May 2021.

Not yet released will be fixed in a react update soon. Issue: https://github.com/facebook/create-react-app/issues/10474 PR: https://github.com/facebook/react/pull/20831

componentWillReceiveProps has been renamed, and is not recommended for use Resolved and published in Alpha PR: https://github.com/oliviertassinari/react-swipeable-views/pull/616

Checklist

• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [x] All new and existing tests passed.
• [ ] The title of my pull request is a short description of the requested changes.

[rizwan2000rm]

@Abhishek-kumar09 Please review this PR and also look into soft shadows theming options. I was not able to figure it out. But commenting it out removed the error from the console.

[ADRE9]

Hey @rizwan2000rm I mean I was also debugging the errors how did you find out the specific location to the errors from the chunk files.

[Abhishek-kumar09]

Github, rebase

[rizwan2000rm]

@Abhishek-kumar09 I think that didn't work. Do you want me to rebase to a single commit ?

[Abhishek-kumar09]

I am seeing that you havn't added anything for removing console errors, I could just see the eslint fixes done with npm run lint:fix. I could see many errors in the console.

[rizwan2000rm]

I'm sorry, I have confirmed with my fork again with latest changes, I don't see any of these errors. I have resolved them. All of those saying "X cannot appear as descendant of p" were due to the following line ` {value === index && (

{children}

  )}`

at line 111, src/pages/CoursePageViewWithVideo/index.js

{value === index && <Box>{children}</Box>} which were resolved just by changing it to this. Typography uses p as a default tag which was wrapped around all those tags, hence resulting in so many errors. All other errors are due to some dependencies installed look above messge for details.

[Abhishek-kumar09]

Looks good, Ok to merge once CI passed