[Closes #143] Resolve `defaultProp` and required prop warnings - Githubissues

sfbrigade / sf-lifeline

GNU Affero General Public License v3.0

7 stars 4 forks source link

[Closes #143] Resolve `defaultProp` and required prop warnings #159

Closed samau3 closed 3 weeks ago

samau3 commented 3 weeks ago

Fixes #143.

There were two Chrome console warnings:

Using defaultProps in the Header.jsx component
Missing prop value despite it being required for the ProtectedRoute component

To resolve these warnings:

defaultProps was removed and a default value is used in the Header.jsx component
Utilized conditional rendering to show a Loader component while user context is being updated