- {['loaded', 'denied'].includes(courseStatus) && (
+ {(courseStatus === LOADED || courseStatus === DENIED) && (
<>
- {toastContent?.message}
+ {toastContent?.message ?? ''}
{metadataModel === 'courseHomeMeta' && ()}
>
@@ -67,16 +80,23 @@ const TabPage = (props) => {
- {courseStatus === 'loading' && (
+ {courseStatus === LOADING && (
)}
- {['loaded', 'denied'].includes(courseStatus) && (
-
+ {(courseStatus === LOADED || courseStatus === DENIED) && courseId && (
+
+ {children}
+
)}
{/* courseStatus 'failed' and any other unexpected course status. */}
- {(!['loading', 'loaded', 'denied'].includes(courseStatus)) && (
+ {courseStatus !== LOADING && courseStatus !== LOADED && courseStatus !== DENIED && (
{errorMessage || intl.formatMessage(messages.failure)}
@@ -86,17 +106,4 @@ const TabPage = (props) => {
);
};
-TabPage.defaultProps = {
- courseId: null,
- unitId: null,
-};
-
-TabPage.propTypes = {
- activeTabSlug: PropTypes.string.isRequired,
- courseId: PropTypes.string,
- courseStatus: PropTypes.string.isRequired,
- metadataModel: PropTypes.string.isRequired,
- unitId: PropTypes.string,
-};
-
export default TabPage;