update proptypes

Author: emily-yang

src/components/lesson/Lesson.js

@@ -13,10 +13,16 @@ const Lesson = ({ lesson, course_size, base_path }) => (
 );
 
 Lesson.propTypes = {
-  title: PropTypes.string.isRequired,
-  src: PropTypes.string.isRequired,
-  description: PropTypes.string.isRequired,
-  order: PropTypes.number.isRequired,
+  lesson: PropTypes.shape({
+    title: PropTypes.string.isRequired,
+    src: PropTypes.string.isRequired,
+    description: PropTypes.string.isRequired,
+    order: PropTypes.number.isRequired,
+    yt_chan_name: PropTypes.string.isRequired,
+    yt_chan_src: PropTypes.string.isRequired,
+    yt_title: PropTypes.string.isRequired,
+    yt_desc: PropTypes.string.isRequired,
+  }).isRequired,
   course_size: PropTypes.number.isRequired,
   base_path: PropTypes.string.isRequired,
 };

src/components/lesson/LessonInfo.js

@@ -16,29 +16,37 @@ const LessonInfo = ({ lesson, base_path, course_size }) => {
       <hr />
       <p id="lesson-description">{description || 'No description provided'}</p>
       <hr />
+
       <div id="yt-info">
-        {/* link to YT chan, original video name, desc */}
         <h3 className="subheader">Youtube credits</h3>
         <p id="yt-chan">
           <span className="title">Creator: </span>
-          <a href={yt_chan_src} target="_blank" rel="noopener noreferrer">{yt_chan_name || 'YouTube Channel'}</a>
+          <a href={yt_chan_src} target="_blank" rel="noopener noreferrer">{yt_chan_name || 'YouTube Channel goes here'}</a>
         </p>
         <p id="yt-title">
           <span className="title">Video title: </span>
           <a href={getVideoURL(src)} target="_blank" rel="noopener noreferrer">{yt_title || 'Original YouTube Video'}</a>
         </p>
         <p id="yt-desc">
           <span className="title">Video Description: </span>
-          {yt_desc || 'YouTube description'}
+          {yt_desc || 'YouTube description goes here'}
         </p>
       </div>
     </div>
   );
 };
+
 LessonInfo.propTypes = {
-  title: PropTypes.string.isRequired,
-  description: PropTypes.string.isRequired,
-  order: PropTypes.number.isRequired,
+  lesson: PropTypes.shape({
+    title: PropTypes.string.isRequired,
+    src: PropTypes.string.isRequired,
+    description: PropTypes.string.isRequired,
+    order: PropTypes.number.isRequired,
+    yt_chan_name: PropTypes.string.isRequired,
+    yt_chan_src: PropTypes.string.isRequired,
+    yt_title: PropTypes.string.isRequired,
+    yt_desc: PropTypes.string.isRequired,
+  }).isRequired,
   base_path: PropTypes.string.isRequired,
   course_size: PropTypes.number.isRequired,
 };

src/components/lesson/LessonLink.js

@@ -41,7 +41,7 @@ const LessonLink = ({ title, length, order, active, base_path }) => {
 LessonLink.propTypes = {
   title: PropTypes.string.isRequired,
   order: PropTypes.number.isRequired,
-  length: PropTypes.number.isRequired,
+  length: PropTypes.string.isRequired,
   active: PropTypes.bool.isRequired,
   base_path: PropTypes.string.isRequired,
 };

src/components/lesson/LessonsPane.js

@@ -27,7 +27,7 @@ LessonsPane.propTypes = {
     PropTypes.shape({
       order: PropTypes.number.isRequired,
       title: PropTypes.string.isRequired,
-      length: PropTypes.number.isRequired,
+      length: PropTypes.string.isRequired,
     }),
   ).isRequired,
   base_path: PropTypes.string.isRequired,

src/components/pages/LessonPage.js

@@ -6,7 +6,6 @@ import '../../css/pages/LessonPage.scss';
 function LessonPage(props) {
   const { curr_lesson_num, lessons, base_path, course_title } = props;
   // const { title, src, description, id } = lessons.find(elem => elem.id === activeId);
-  // const { title, src, description, order } = lessons[curr_lesson_num];
   const lesson = lessons[curr_lesson_num];
   const total = lessons.length;
 
@@ -31,7 +30,7 @@ LessonPage.propTypes = {
       order: PropTypes.number.isRequired,
       title: PropTypes.string.isRequired,
       src: PropTypes.string.isRequired,
-      length: PropTypes.number.isRequired,
+      length: PropTypes.string.isRequired,
       description: PropTypes.string.isRequired,
     }),
   ).isRequired,