Revert "refactor: replace defaultProps with default parameters"

This reverts commit 50688d5.

Author: mraman-2U

src/courseware/course/Course.jsx

@@ -19,9 +19,9 @@ import { CourseOutlineMobileSidebarTriggerSlot } from '../../plugin-slots/Course
 import { CourseBreadcrumbsSlot } from '../../plugin-slots/CourseBreadcrumbsSlot';
 
 const Course = ({
-  courseId = null,
-  sequenceId = null,
-  unitId = null,
+  courseId,
+  sequenceId,
+  unitId,
   nextSequenceHandler,
   previousSequenceHandler,
   unitNavigationHandler,
@@ -137,6 +137,12 @@ Course.propTypes = {
   windowWidth: PropTypes.number.isRequired,
 };
 
+Course.defaultProps = {
+  courseId: null,
+  sequenceId: null,
+  unitId: null,
+};
+
 const CourseWrapper = (props) => {
   // useWindowSize initially returns an undefined width intentionally at first.
   // See https://www.joshwcomeau.com/react/the-perils-of-rehydration/ for why.

src/generic/user-messages/UserMessagesProvider.jsx

@@ -29,7 +29,7 @@ function popFlashMessages() {
   return popLocalStorage(FLASH_MESSAGES_LOCAL_STORAGE_KEY) || [];
 }
 
-const UserMessagesProvider = ({ children = null }) => {
+const UserMessagesProvider = ({ children }) => {
   // Note: The callbacks (add, remove, clear) below interact with useState in very subtle ways.
   // When we call setMessages, we always do so with the function-based form of the handler, making
   // use of the "current" state and not relying on lexical scoping to access the state exposed
@@ -102,4 +102,8 @@ UserMessagesProvider.propTypes = {
   children: PropTypes.node,
 };
 
+UserMessagesProvider.defaultProps = {
+  children: null,
+};
+
 export default UserMessagesProvider;