book: retire noHeader prop

Author: tima101

book/4-end/lib/withAuth.js

@@ -65,4 +65,4 @@ export default (Page, { loginRequired = true, logoutRequired = false } = {}) =>
 
       return <Page {...this.props} />;
     }
-};
+  };

book/7-end/lib/withAuth.js

@@ -6,8 +6,9 @@ let globalUser = null;
 
 export default (
   Page,
-  { loginRequired = true, logoutRequired = false, adminRequired = false, noHeader = false } = {},
-) => class BaseComponent extends React.Component {
+  { loginRequired = true, logoutRequired = false, adminRequired = false } = {},
+) =>
+  class BaseComponent extends React.Component {
     static propTypes = {
       user: PropTypes.shape({
         id: PropTypes.string,
@@ -49,7 +50,7 @@ export default (
         user._id = user._id.toString();
       }
 
-      const props = { user, isFromServer, noHeader };
+      const props = { user, isFromServer };
 
       if (Page.getInitialProps) {
         Object.assign(props, (await Page.getInitialProps(ctx)) || {});

book/7-end/pages/_app.js

@@ -37,7 +37,6 @@ class MyApp extends App {
     const { Component, pageProps } = this.props;
 
     // console.log(pageProps);
-    // console.log(pageProps.noHeader);
 
     return (
       <Container>
@@ -46,7 +45,7 @@ class MyApp extends App {
         <ThemeProvider theme={theme}>
           {/* CssBaseline kickstart an elegant, consistent, and simple baseline to build upon. */}
           <CssBaseline />
-          {pageProps.noHeader ? null : <Header {...pageProps} />}
+          {pageProps.chapter ? null : <Header {...pageProps} />}
           <Component {...pageProps} />
           <Notifier />
         </ThemeProvider>

book/7-end/pages/public/read-chapter.js

@@ -328,4 +328,4 @@ class ReadChapter extends React.Component {
   }
 }
 
-export default withAuth(ReadChapter, { loginRequired: false, noHeader: true });
+export default withAuth(ReadChapter, { loginRequired: false });

book/8-begin/lib/withAuth.js

@@ -6,7 +6,7 @@ let globalUser = null;
 
 export default (
   Page,
-  { loginRequired = true, logoutRequired = false, adminRequired = false, noHeader = false } = {},
+  { loginRequired = true, logoutRequired = false, adminRequired = false } = {},
 ) =>
   class BaseComponent extends React.Component {
     static propTypes = {
@@ -50,7 +50,7 @@ export default (
         user._id = user._id.toString();
       }
 
-      const props = { user, isFromServer, noHeader };
+      const props = { user, isFromServer };
 
       if (Page.getInitialProps) {
         Object.assign(props, (await Page.getInitialProps(ctx)) || {});

book/8-begin/pages/_app.js

@@ -37,7 +37,6 @@ class MyApp extends App {
     const { Component, pageProps } = this.props;
 
     // console.log(pageProps);
-    // console.log(pageProps.noHeader);
 
     return (
       <Container>
@@ -46,7 +45,7 @@ class MyApp extends App {
         <ThemeProvider theme={theme}>
           {/* CssBaseline kickstart an elegant, consistent, and simple baseline to build upon. */}
           <CssBaseline />
-          {pageProps.noHeader ? null : <Header {...pageProps} />}
+          {pageProps.chapter ? null : <Header {...pageProps} />}
           <Component {...pageProps} />
           <Notifier />
         </ThemeProvider>

book/8-begin/pages/public/read-chapter.js

@@ -21,7 +21,7 @@ class ReadChapter extends React.Component {
   static propTypes = {
     chapter: PropTypes.shape({
       _id: PropTypes.string.isRequired,
-      isPurchased: PropTypes.bool.isRequired,
+      isPurchased: PropTypes.bool,
       isFree: PropTypes.bool.isRequired,
       htmlContent: PropTypes.string,
       htmlExcerpt: PropTypes.string,
@@ -330,4 +330,4 @@ class ReadChapter extends React.Component {
   }
 }
 
-export default withAuth(ReadChapter, { loginRequired: false, noHeader: true });
+export default withAuth(ReadChapter, { loginRequired: false });

book/8-end/lib/withAuth.js

@@ -6,8 +6,9 @@ let globalUser = null;
 
 export default (
   Page,
-  { loginRequired = true, logoutRequired = false, adminRequired = false, noHeader = false } = {},
-) => class BaseComponent extends React.Component {
+  { loginRequired = true, logoutRequired = false, adminRequired = false } = {},
+) =>
+  class BaseComponent extends React.Component {
     static propTypes = {
       user: PropTypes.shape({
         id: PropTypes.string,
@@ -49,7 +50,7 @@ export default (
         user._id = user._id.toString();
       }
 
-      const props = { user, isFromServer, noHeader };
+      const props = { user, isFromServer };
 
       if (Page.getInitialProps) {
         Object.assign(props, (await Page.getInitialProps(ctx)) || {});

book/8-end/pages/_app.js

@@ -37,7 +37,6 @@ class MyApp extends App {
     const { Component, pageProps } = this.props;
 
     // console.log(pageProps);
-    // console.log(pageProps.noHeader);
 
     return (
       <Container>
@@ -46,7 +45,7 @@ class MyApp extends App {
         <ThemeProvider theme={theme}>
           {/* CssBaseline kickstart an elegant, consistent, and simple baseline to build upon. */}
           <CssBaseline />
-          {pageProps.noHeader ? null : <Header {...pageProps} />}
+          {pageProps.chapter ? null : <Header {...pageProps} />}
           <Component {...pageProps} />
           <Notifier />
         </ThemeProvider>

book/8-end/pages/public/read-chapter.js

@@ -23,7 +23,7 @@ class ReadChapter extends React.Component {
   static propTypes = {
     chapter: PropTypes.shape({
       _id: PropTypes.string.isRequired,
-      isPurchased: PropTypes.bool.isRequired,
+      isPurchased: PropTypes.bool,
       isFree: PropTypes.bool.isRequired,
       htmlContent: PropTypes.string,
       htmlExcerpt: PropTypes.string,
@@ -35,11 +35,13 @@ class ReadChapter extends React.Component {
       asPath: PropTypes.string.isRequired,
     }).isRequired,
     showStripeModal: PropTypes.bool.isRequired,
+    noHeader: PropTypes.bool,
   };
 
   static defaultProps = {
     chapter: null,
     user: null,
+    noHeader: true,
   };
 
   constructor(props, ...args) {
@@ -353,5 +355,4 @@ class ReadChapter extends React.Component {
 
 export default withAuth(withRouter(ReadChapter), {
   loginRequired: false,
-  noHeader: true,
 });